Skip to content

U/kertai/automatic update#61

Open
mkertai wants to merge 1 commit into
masterfrom
u/kertai/automatic-update
Open

U/kertai/automatic update#61
mkertai wants to merge 1 commit into
masterfrom
u/kertai/automatic-update

Conversation

@mkertai

@mkertai mkertai commented May 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Add a workflow_dispatch-triggered GitHub Actions workflow that:
- Updates requirements*.txt via `make update-requirements`
- Bumps the version via `make bump-version`
- Commits and pushes to an `auto-update-requirements` branch
- Calls ci.yml via workflow_call to run tests on the updated code
- If CI passes: merges to master and deletes the branch
- If CI fails: deletes the branch only

Also update ci.yml to:
- Add workflow_call trigger with optional ref input so it can be
  called from update_requirements with the correct commit sha
- Add auto-update-requirements to push trigger branches
- Upgrade actions/checkout from v3 to v4

References: azure #710347
@mkertai mkertai force-pushed the u/kertai/automatic-update branch from b5d4559 to ea55bda Compare May 21, 2026 15:54

@abajusz23 abajusz23 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think the commit ea55bda should be merged together with the update of types-setuptools. Because the linter wont let you merge anyway.

@davedusa davedusa left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why is the pin charset-normalizer>=3.4.7 commit needed?
Could you explain in the commit message?

@mkertai mkertai force-pushed the u/kertai/automatic-update branch from ea55bda to d33b681 Compare June 16, 2026 11:45
@adamseres adamseres self-requested a review July 3, 2026 09:35
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.

4 participants