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

Fix pylint? #130

Merged
merged 5 commits into from
Jun 22, 2024
Merged

Fix pylint? #130

merged 5 commits into from
Jun 22, 2024

Conversation

lilyminium
Copy link
Member

@lilyminium lilyminium commented Jun 22, 2024

Fixes #125

Changes made in this Pull Request:

  • Jobs are now manually specified in CI to avoid caching errors.

PR Checklist

  • Tests?
  • Docs?
  • CHANGELOG.md updated?
  • AUTHORS.md updated?
  • Issue raised/referenced?

@lilyminium lilyminium marked this pull request as ready for review June 22, 2024 08:44
@lilyminium
Copy link
Member Author

Ok, manually specifying the jobs seems to have fixed it. It does mean that adding new jobs to cookie CI needs to manually add the check to the action workflow as well, which is suboptimal. Raised #131 to look at this in the future.

@lilyminium lilyminium merged commit 4f0fd13 into main Jun 22, 2024
12 checks passed
@lilyminium lilyminium deleted the re-do-all-tests branch June 22, 2024 08:56
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.

pylint check failing in CI
1 participant