-
Notifications
You must be signed in to change notification settings - Fork 107
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
Feature/ghcr integration #217
base: master
Are you sure you want to change the base?
Feature/ghcr integration #217
Conversation
* fixing breaking test * fixing test to search for Type anywhere in the header * fixing task list test * fixing test * fixed one test but broke another * grouping tests to stop leaking state between tests * moving tests to make them run more consistently * making test less flakey * changing test to make more stable * making test more stable * commenting out test. will be fixed as part of history-virtualize-scroll
* fixing breaking test * fixing test to search for Type anywhere in the header * fixing task list test * fixing test * fixed one test but broke another * grouping tests to stop leaking state between tests * moving tests to make them run more consistently * making test less flakey * changing test to make more stable * making test more stable * commenting out test. will be fixed as part of history-virtualize-scroll
@@ -22,6 +22,22 @@ jobs: | |||
with: | |||
username: ${{ secrets.CADENCE_WEB_DOCKERHUB_USERNAME }} | |||
password: ${{ secrets.CADENCE_WEB_DOCKERHUB_TOKEN }} | |||
repository: ubercadence/web | |||
repository: justatuber/web |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to swap back to ubercadence/web
uses: docker/build-push-action@v1 | ||
with: | ||
username: ${{ github.actor }} | ||
password: ${{ secrets.CADENCE_WEB_GHCR_PAT }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to change back to GITHUB_TOKEN
once github supports this.
See here for more information
username: ${{ github.actor }} | ||
password: ${{ secrets.CADENCE_WEB_GHCR_PAT }} | ||
registry: ghcr.io | ||
repository: just-at-uber/cadence-web/web |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to update repository to uber/cadence-web/web
once organization has been initialized with ghcr (see here).
Experimenting with integrating with ghcr for docker image uploads.
Some caveats:
PAT
token instead ofGITHUB_TOKEN
. github is aware of issue and working on integratingGITHUB_TOKEN
(see here)GITHUB_TOKEN
is fixed.Screenshots
just-at-uber release
just-at-uber packages