Skip to content

Renovate version maintenance for Autorest client generation#699

Merged
aldbr merged 3 commits into
DIRACGrid:mainfrom
AcquaDiGiorgio:issue-523-autorest-update-maintenance-task
Feb 16, 2026
Merged

Renovate version maintenance for Autorest client generation#699
aldbr merged 3 commits into
DIRACGrid:mainfrom
AcquaDiGiorgio:issue-523-autorest-update-maintenance-task

Conversation

@AcquaDiGiorgio

@AcquaDiGiorgio AcquaDiGiorgio commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

This, at this current moment, only affects the verison stored at the variables AUTOREST_VERSION, AUTOREST_CORE_VERSION and AUTOREST_PUGINS in dirac-testing/src/diracx/testing/client_generation.py.

I don't know how to add autorest as a dependency in the pixi.toml because autorest is an npm package and it is not available on the anaconda channels.

Tested it using Mend Renovate. At my Fork you will be able to see some Closed PRs created by the bot. PRs #7, #8, #9 and #10 are all related to Autorest.

See #523

@AcquaDiGiorgio AcquaDiGiorgio marked this pull request as draft November 5, 2025 15:54
@fstagni fstagni linked an issue Nov 11, 2025 that may be closed by this pull request
3 tasks
@aldbr

aldbr commented Feb 5, 2026

Copy link
Copy Markdown
Contributor

@AcquaDiGiorgio what's the status of this PR? Do you need anything from us here?
Seems like it would be a good feature!

@AcquaDiGiorgio

Copy link
Copy Markdown
Contributor Author

I need to know how to add autorest as a dependency in the pixi.toml, as I mentioned in the comment.

After that clarification, it should be ready to review / merge

@AcquaDiGiorgio AcquaDiGiorgio marked this pull request as ready for review February 9, 2026 14:50

@aldbr aldbr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Other than my minor comment, I think it's the most pragmatic way of bumping the version of autorest.

Comment thread renovate.json Outdated
@DIRACGridBot DIRACGridBot marked this pull request as draft February 13, 2026 08:41
@aldbr

aldbr commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

Can you rebase the branch please?

@AcquaDiGiorgio AcquaDiGiorgio force-pushed the issue-523-autorest-update-maintenance-task branch from 9012450 to 7e415f1 Compare February 13, 2026 09:59
@aldbr aldbr marked this pull request as ready for review February 13, 2026 10:06

@aldbr aldbr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great, I think I just have one last comment: could you try to group these updates within a same PR please? I think it would make sense, and I guess you would need to define a packageRules entry for that.

@DIRACGridBot DIRACGridBot marked this pull request as draft February 13, 2026 10:35
@AcquaDiGiorgio

Copy link
Copy Markdown
Contributor Author

I finally was able to group the dependencies in a package rule, see #28

@aldbr aldbr marked this pull request as ready for review February 16, 2026 10:45
@aldbr aldbr enabled auto-merge (squash) February 16, 2026 10:49
@aldbr aldbr merged commit 9fc4231 into DIRACGrid:main Feb 16, 2026
26 checks passed
Stellatsuu pushed a commit to Stellatsuu/diracx that referenced this pull request Mar 10, 2026
…IRACGrid#699)

* chore(renovate): renovate.json for Autorest versioning at client generation

* chore(renovate): Group autorest package updates
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.

Add maintenance task to update versions of autorest

2 participants