Skip to content

Provide info output for potential missing permissions block#73

Merged
lucperkins merged 1 commit intomainfrom
clarify-log-message
Jul 16, 2024
Merged

Provide info output for potential missing permissions block#73
lucperkins merged 1 commit intomainfrom
clarify-log-message

Conversation

@lucperkins
Copy link
Copy Markdown
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
Copy Markdown
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 enabled auto-merge July 16, 2024 00:41
@lucperkins lucperkins merged commit 56ad602 into main Jul 16, 2024
@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`)
detsys-pr-bot pushed a commit to detsys-pr-bot/magic-nix-cache-action that referenced this pull request Mar 19, 2025
…m DeterminateSystems/dependabot/github_actions/actions-deps-533f70582a` (`bbdef961988d15c82ae95b390c07c035a24a2c98`)
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