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

Add GH secret for API key to try-runtime job #757

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

simonsso
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jun 22, 2023

Codecov Report

Merging #757 (c8f36b6) into master (55e44ba) will not change coverage.
The diff coverage is n/a.

❗ Current head c8f36b6 differs from pull request most recent head 5b3eeb4. Consider uploading reports for the commit 5b3eeb4 to get more accurate results

@@           Coverage Diff           @@
##           master     #757   +/-   ##
=======================================
  Coverage   59.00%   59.00%           
=======================================
  Files          38       38           
  Lines        4710     4710           
=======================================
  Hits         2779     2779           
  Misses       1931     1931           

@simonsso simonsso force-pushed the chore/20230621-try-runtime-ci branch 4 times, most recently from c8f36b6 to 63f03ca Compare June 22, 2023 09:06
@@ -46,7 +46,7 @@ jobs:
if: github.event_name != 'pull_request'
uses: docker/login-action@v2
with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
username: ${{ vars.DOCKERHUB_USERNAME }}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved username from secrets so it can be printed

@simonsso
Copy link
Contributor Author

Looks good to me

@simonsso simonsso requested review from Hounsette and a team June 22, 2023 10:21
Add GH secret for API key to try-runtime job
Move DockerHub username from secret to vars
@simonsso simonsso force-pushed the chore/20230621-try-runtime-ci branch from 63f03ca to 5b3eeb4 Compare June 22, 2023 10:31
@simonsso simonsso merged commit 1c64241 into master Jun 22, 2023
@simonsso simonsso deleted the chore/20230621-try-runtime-ci branch June 22, 2023 10:38
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.

1 participant