Skip to content

Add support for installing pipelines-agent binaries, but default to installing vsts-agent binaries#43

Merged
jwittner merged 2 commits into
microsoft:developfrom
tristanbarcelon:develop
Aug 15, 2023
Merged

Add support for installing pipelines-agent binaries, but default to installing vsts-agent binaries#43
jwittner merged 2 commits into
microsoft:developfrom
tristanbarcelon:develop

Conversation

@tristanbarcelon

@tristanbarcelon tristanbarcelon commented Aug 3, 2023

Copy link
Copy Markdown
Contributor

binaries come in vsts-agent and pipelines-agent flavors. Former has a dependency on Node6 while latter supports Node10 or later. For users of this module who want to switch to modern tasks that can run on Node10 or newer, they can set AgentType parameter to pipelines-agent and it will pick the matching resource in Find-VstsAgent.

Resolves #42

… but default to installing vsts-agent binaries as it did prior

@jwittner jwittner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great changes! Just a question and one suggestion. Thanks so much for the contribution!

Comment thread VSTSAgent/VSTSAgent.psm1
Comment thread VSTSAgent/VSTSAgent.psm1 Outdated

@tristanbarcelon tristanbarcelon left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @jwittner, I've submitted changes per your request. Are you ok to approve the PR?

@jwittner jwittner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great!

@jwittner
jwittner merged commit d48661e into microsoft:develop Aug 15, 2023
@jwittner

jwittner commented Aug 15, 2023

Copy link
Copy Markdown
Member

@tristanbarcelon - available on the PSGallery as 2.0.20-develop. Thanks a ton for the great contribution!

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.

Support installation of node10-based pipelines-agent in lieu of vsts-agent

2 participants