Skip to content

Automatic PRs to update xhermes/boutdata/xbout#582

Merged
mikekryjak merged 6 commits into
masterfrom
auto-update-pydeps
Jun 25, 2026
Merged

Automatic PRs to update xhermes/boutdata/xbout#582
mikekryjak merged 6 commits into
masterfrom
auto-update-pydeps

Conversation

@mikekryjak

@mikekryjak mikekryjak commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

The dependencies are installed from a txt list, and there is a bot to update the list at 6am every morning.
Not done this before, so any comments/criticism welcome!

Instead of using dependabot, this uses a simpler approach through https://github.com/peter-evans/create-pull-request.

@mikekryjak mikekryjak marked this pull request as draft June 16, 2026 10:14
@mikekryjak mikekryjak force-pushed the auto-update-pydeps branch from 6428a8c to 5eb04bb Compare June 16, 2026 10:34
@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.08%. Comparing base (c8aa796) to head (3155c60).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #582      +/-   ##
==========================================
- Coverage   49.32%   49.08%   -0.25%     
==========================================
  Files          96       96              
  Lines       10029    10038       +9     
  Branches     1453     1452       -1     
==========================================
- Hits         4947     4927      -20     
- Misses       4587     4606      +19     
- Partials      495      505      +10     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dschwoerer

Copy link
Copy Markdown
Collaborator

Why does codecov think this reduces coverage?

@dschwoerer dschwoerer marked this pull request as ready for review June 19, 2026 10:55
@dschwoerer dschwoerer changed the title Attempt at automatic PRs to update xhermes/boutdata/xbout Automatic PRs to update xhermes/boutdata/xbout Jun 19, 2026

@dschwoerer dschwoerer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looking forward to testing this in action!

@ZedThree

Copy link
Copy Markdown
Member

If we put these dependencies in the pyproject.toml, could dependabot just keep them up-to-date for us?

@mikekryjak

Copy link
Copy Markdown
Collaborator Author

I don't have experience with dependabot. My stochastic parrot says that it would be possible with sufficient config, but it wouldn't be as clean.

In this system, there is only one PR updating xBOUT, xHermes and boutdata in one. You have more experience in this than me, so if you think dependabot is better, we could have a go... but maybe we can merge this in the meantime so that we have something in place.

@mikekryjak mikekryjak merged commit 2ea120b into master Jun 25, 2026
5 of 6 checks passed
@mikekryjak mikekryjak deleted the auto-update-pydeps branch June 25, 2026 12:49
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.

3 participants