Skip to content

Failed to publish 0.5.1 #187

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

Closed
nicolasstucki opened this issue Jan 20, 2021 · 3 comments · Fixed by #188
Closed

Failed to publish 0.5.1 #187

nicolasstucki opened this issue Jan 20, 2021 · 3 comments · Fixed by #188

Comments

@nicolasstucki
Copy link
Contributor

See https://github.com/scala/vscode-scala-syntax/runs/1733572597?check_suite_focus=true

@nicolasstucki
Copy link
Contributor Author

@MaximeKjaer did you see a similar error before?

@nicolasstucki
Copy link
Contributor Author

I received an email from GitHub after this saying

Basic authentication using a password to Git is deprecated and will soon no longer work. Visit https://github.blog/2020-12-15-token-authentication-requirements-for-git-operations/ for more information around suggested workarounds and removal dates.

@MaximeKjaer
Copy link
Contributor

No, haven't seen this before. I am able to replicate locally.

It seems that github-changes has released a major version recently, which included a refactor that probably introduced a bug. The lesson here is that we shouldn't use npx in CI scripts, as major releases risk breaking our CI. I'll submit a PR that pins the version of the dependency; this seems to fix the problem locally.

MaximeKjaer added a commit to MaximeKjaer/vscode-scala-syntax that referenced this issue Jan 20, 2021
MaximeKjaer added a commit to MaximeKjaer/vscode-scala-syntax that referenced this issue Jan 20, 2021
@nicolasstucki nicolasstucki linked a pull request Jan 20, 2021 that will close this issue
nicolasstucki added a commit that referenced this issue Jan 20, 2021
PanAeon pushed a commit to PanAeon/vscode-scala-syntax that referenced this issue Apr 28, 2022
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 a pull request may close this issue.

2 participants