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

[github-actions] Update CI workflow to avoid login issues #182

Merged
merged 1 commit into from
May 5, 2021

Conversation

jstockwin
Copy link
Owner

Description

Some PRs from other authors are failing CI due to not authenticating with the docker repo. This was needed as we used the "build with cache", which pushed and pulled partial images. I have now removed this.

Linked issues

None

Testing

CI will pass

Checklist

  • I have provided a good description of the change above
  • I have added any necessary tests
  • I have added all necessary type hints
  • I have checked my linting (docker-compose run --rm lint)
  • I have added/updated all necessary documentation
  • I have updated CHANGELOG.md, following the format from
    Keep a Changelog.

@jstockwin jstockwin merged commit 5c53291 into master May 5, 2021
@jstockwin jstockwin deleted the github-actions-fix branch May 5, 2021 09:43
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.

2 participants