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

deno publish --dry-run requires --token in pre-commit hook #22768

Closed
MarkTiedemann opened this issue Mar 7, 2024 · 1 comment
Closed

deno publish --dry-run requires --token in pre-commit hook #22768

MarkTiedemann opened this issue Mar 7, 2024 · 1 comment
Labels
bug Something isn't working correctly publish Related to "deno publish" subcommand

Comments

@MarkTiedemann
Copy link
Contributor

I'm running deno publish --dry-run in a deno task in a pre-commit hook.

When I run the deno task directly, this works. When it is run by the hook, however, it fails with the following error:

error: No means to authenticate. Pass a token to `--token`.

I feel like you should never need to pass --token when using --dry-run.

Currently, I work-around this issue by adding a fake token, e. g. deno publish --dry-run --token=dry-run


Deno 1.41.1, Windows 11

@lucacasonato lucacasonato added bug Something isn't working correctly publish Related to "deno publish" subcommand labels Mar 7, 2024
@lucacasonato
Copy link
Member

Duplicate of #22658

It's fixed in canary already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working correctly publish Related to "deno publish" subcommand
Projects
None yet
Development

No branches or pull requests

2 participants