Improve CI automation#667
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #667 +/- ##
=======================================
Coverage 87.63% 87.63%
=======================================
Files 8 8
Lines 833 833
=======================================
Hits 730 730
Misses 103 103 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4811259a93
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| permissions: | ||
| contents: write | ||
| pull-requests: write |
There was a problem hiding this comment.
Grant Actions permission for workflow update PRs
When the new Dependabot github-actions group opens a non-major update, the PR changes files under .github/workflows, but this job enables auto-merge with a GITHUB_TOKEN that only has contents and pull-requests permissions. The GitHub CLI auto-merge call fails in that scenario with the workflow-file permission error (same gh pr merge --auto --squash/permissions combination is documented in cli/cli#11493), so action-update PRs will never get auto-merge enabled unless this adds actions: write or excludes github-actions updates from this workflow.
Useful? React with 👍 / 👎.
|




Motivation
Description
.github/dependabot.ymlto loweropen-pull-requests-limit, add dependency labels and group patterns for GitHub Actions and pip updates..github/labels.ymlfordependencies,python, andgithub_actionsto match Dependabot labeling..github/workflows/auto-merge-dependabot.ymlto auto-merge non-major Dependabot updates when safe..github/workflows/pre-commit.ymland updated.github/workflows/pytest.ymland.github/workflows/documentation.ymlto enableconcurrency, add pip caching, and upload coverage artifacts per matrix job.Testing
pythonandyaml.safe_load, which completed successfully.pytest --cov=src/pyecotrend_ista --cov-report=xml --cov-report=term-missing tests, which produced444 passedand wrotecoverage.xml(total coverage reported ~88%).tox -e py311was skipped because the container does not provide apy311interpreter.tox -e pre-commitcould not complete due to an environmentnpmerror (SELF_SIGNED_CERT_IN_CHAIN) while installing theprettierpre-commit dependency.Codex Task