Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide info output for potential missing permissions block #73

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

lucperkins
Copy link
Member

Both the MNC binary itself and this Action have subtly different log output when FlakeHub Cache is enabled via use-flakehub: true but then disabled by a missing token. This PR should provide a bit more clarity around where the issue is coming from and also suggests a fix.

Copy link
Member

@colemickens colemickens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤗 nice

@lucperkins lucperkins merged commit 56ad602 into main Jul 16, 2024
7 checks passed
@lucperkins lucperkins deleted the clarify-log-message branch July 16, 2024 00:43
detsys-pr-bot pushed a commit to detsys-pr-bot/magic-nix-cache-action that referenced this pull request Aug 1, 2024
…m DeterminateSystems/clarify-log-message` (`d353465ae6a55761963005617a7780f2bf7e4ec2`)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants