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

Update publish action auth #224

Merged
merged 14 commits into from
Oct 7, 2021
Merged

Update publish action auth #224

merged 14 commits into from
Oct 7, 2021

Conversation

mofojed
Copy link
Member

@mofojed mofojed commented Oct 7, 2021

Previously was authenticating as mofojed, which is my personal account. Switched to my mikebender account, which uses mikebender@deephaven.io email address and does have access to publish @deephaven packages.
Also added the --no-verify-access flag, because that breaks publish when using an Automation token from CI: lerna/lerna#2788

@mofojed mofojed added this to the October 2021 milestone Oct 7, 2021
@mofojed mofojed requested a review from devinrsmith October 7, 2021 12:23
@mofojed mofojed self-assigned this Oct 7, 2021
I've locked myself out of my mikebender account :/
until I get my mikebender account sorted, this is the way to go
Now that I've got auth tokens sorted out I think...
@mofojed mofojed changed the title Fix publish auth Update publish action names Oct 7, 2021
We use NPM automation tokens. Apparently you can't check the user endpoint for that, which is causing publish to fail.
Add --no-verify-access to try and publish without verifying access - if you don't have access, the publish is going to fail anyway.
lerna/lerna#2788
@mofojed mofojed changed the title Update publish action names Update publish action auth Oct 7, 2021
@mofojed mofojed merged commit e9dfa39 into main Oct 7, 2021
@mofojed mofojed deleted the fix-publish-auth branch October 7, 2021 15:51
mofojed added a commit to mofojed/web-client-ui that referenced this pull request Oct 12, 2021
* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [deephaven#203](deephaven#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))

* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [deephaven#203](deephaven#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))
* `icons`
  * [deephaven#220](deephaven#220) update to latest @vscode/codicons and add dhFreeze icon ([@dsmmcken](https://github.com/dsmmcken))

* `code-studio`
  * [deephaven#225](deephaven#225) Handle invalid layout files from the server correctly ([@mofojed](https://github.com/mofojed))
* `components`, `console`
  * [deephaven#202](deephaven#202) Don't show custom context menu on right-click in Monaco editor ([@vbabich](https://github.com/vbabich))

* [deephaven#226](deephaven#226) Add fetch-depth flag in alpha publish ([@mofojed](https://github.com/mofojed))
* [deephaven#224](deephaven#224) Update publish action auth ([@mofojed](https://github.com/mofojed))

- Don ([@dsmmcken](https://github.com/dsmmcken))
- Mike Bender ([@mofojed](https://github.com/mofojed))
- [@vbabich](https://github.com/vbabich)
@mofojed mofojed mentioned this pull request Oct 12, 2021
mofojed added a commit that referenced this pull request Oct 12, 2021
* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [#203](#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))

* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [#203](#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))
* `icons`
  * [#220](#220) update to latest @vscode/codicons and add dhFreeze icon ([@dsmmcken](https://github.com/dsmmcken))

* `code-studio`
  * [#225](#225) Handle invalid layout files from the server correctly ([@mofojed](https://github.com/mofojed))
* `components`, `console`
  * [#202](#202) Don't show custom context menu on right-click in Monaco editor ([@vbabich](https://github.com/vbabich))

* [#226](#226) Add fetch-depth flag in alpha publish ([@mofojed](https://github.com/mofojed))
* [#224](#224) Update publish action auth ([@mofojed](https://github.com/mofojed))

- Don ([@dsmmcken](https://github.com/dsmmcken))
- Mike Bender ([@mofojed](https://github.com/mofojed))
- [@vbabich](https://github.com/vbabich)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants