Skip to content

RFE: #288#324

Merged
Qalthos merged 9 commits intoansible:mainfrom
dbrennand:patch/#288
May 15, 2025
Merged

RFE: #288#324
Qalthos merged 9 commits intoansible:mainfrom
dbrennand:patch/#288

Conversation

@dbrennand
Copy link
Copy Markdown
Contributor

This PR implements a new option to the ade install command to specify the version of ansible-dev-tools to install.

Example Usage:

ade install --venv .venv --seed --ansible-dev-tools-version 24.4.1
ade install --venv .venv --seed --adtv 24.4.1

I tried to keep the changes similar to what was done in #319. Resolves #288.

@dbrennand dbrennand changed the title Patch/#288 RFE: #288 Apr 26, 2025
@cidrblock
Copy link
Copy Markdown
Collaborator

Can you add adtv to the cspell dictionary?
Can you also add an "info" message that says "Specifying an ansible-dev-tools version may result in installing a version that is no longer supported. Please use the latest release of ansible-dev-tools if you encouter issues or unexpected behaviour"

We should be good to go after that, TY!

@dbrennand
Copy link
Copy Markdown
Contributor Author

Ready @cidrblock 🙂

@dbrennand
Copy link
Copy Markdown
Contributor Author

Passing now @cidrblock 🙂

@shatakshiiii shatakshiiii requested a review from cidrblock May 12, 2025 08:36
@Qalthos Qalthos merged commit 7c6a247 into ansible:main May 15, 2025
20 checks passed
@dbrennand dbrennand deleted the patch/#288 branch May 16, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

RFE: Allow user to specify the version of ADT

3 participants