Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use latest version AzDo replacetokens task #533

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jeroenmaes
Copy link

Update to version 6 of the AzDo replace tokens task.
Version 3 is marked as deprecated.

@vandriot
Copy link
Contributor

Hi,

I think you also need to update some of the input parameter that were renamed in v6 of the tasks.
There is all the breaking changes on the marketplace page: https://marketplace.visualstudio.com/items?itemName=qetza.replacetokens

From a quick review, I can see the following changes that will impact the current pipeline:

  • renamed input targetFiles to sources
  • renamed input writeBOM to addBOM
  • renamed input verbosity to logLevel
    • removed verbosity value off (see new supported values for replacement)
  • renamed input actionOnMissing to missingVarLog
  • replaced keepToken with missingVarAction with value keep

@waelkdouh waelkdouh self-requested a review April 19, 2024 12:33
@waelkdouh
Copy link
Contributor

@jeroenmaes can you please refer to the changes that @vandriot is suggesting. Also any chance you can take a look at the GitHub actions implementation to see if there are similar required changes there as well?

@jeroenmaes
Copy link
Author

@jeroenmaes can you please refer to the changes that @vandriot is suggesting. Also any chance you can take a look at the GitHub actions implementation to see if there are similar required changes there as well?

Will do

@waelkdouh
Copy link
Contributor

Hi we just released v6 release candidate. Can you please try to apply your PR on the new code base?

@felipementel
Copy link

@waelkdouh
Copy link
Contributor

@felipementel what are you suggesting we do?

@felipementel
Copy link

@felipementel what are you suggesting we do?

There are two PR with the same subject
#622 and #533

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