Skip to content

feat: Automatically open PR for new pixi_api releases#56

Merged
haecker-felix merged 3 commits intoprefix-dev:mainfrom
haecker-felix:auto-pr-pixi
Feb 25, 2026
Merged

feat: Automatically open PR for new pixi_api releases#56
haecker-felix merged 3 commits intoprefix-dev:mainfrom
haecker-felix:auto-pr-pixi

Conversation

@haecker-felix
Copy link
Collaborator

Unfortunately a bit tricky to test locally (same with reviewing)....

Copy link
Contributor

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

No inline bash scripts please. Please ask claude to make Python scripts using https://prefix.dev/channels/conda-forge/packages/pygithub wrapped in a Pixi task. Ideally, at least parts of it should be locally runnable

@haecker-felix
Copy link
Collaborator Author

I added a new automation feature, not 100% sure how to feel about that. I tried to be consistent with prefix-dev/pixi, but we don't have this kind kind of automation there.

Copy link
Contributor

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Please use tomlkit instead of regex.

The commit part should be part of a separate pixi task otherwise it's annoying to run locally.

Also make sure to run it locally on your own machine

@CLAassistant
Copy link

CLAassistant commented Feb 17, 2026

CLA assistant check
All committers have signed the CLA.

@haecker-felix
Copy link
Collaborator Author

Also make sure to run it locally on your own machine

pixi run update-pixi is tested and works. create-pixi-update-pr is a bit more tricky to test, especially the github.token part.

The commit part should be part of a separate pixi task otherwise it's annoying to run locally.

it already is. You can run update-pixi without commiting / opening a PR.

Copy link
Contributor

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Thanks, let's check if it works out in practice :)

@haecker-felix haecker-felix enabled auto-merge (squash) February 25, 2026 10:40
@haecker-felix haecker-felix merged commit 5587899 into prefix-dev:main Feb 25, 2026
7 checks passed
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