-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fix DotNetFinalVersionKind variable in yml #170
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's try it.
Didn't work in the internal test run. |
Per offline discussion we should try to make this work with https://learn.microsoft.com/en-us/azure/devops/pipelines/process/runtime-parameters?view=azure-devops&tabs=script#use-parameters-in-pipelines |
There are 3 possible values according to the docs:
I switched to trying using parameters: https://learn.microsoft.com/en-us/azure/devops/pipelines/process/runtime-parameters?view=azure-devops&tabs=script Here's a job testing it: https://dev.azure.com/dnceng/internal/_build/results?buildId=2581675&view=logs&s=6884a131-87da-5381-61f3-d7acc3b91d76 Here's how setting the parameter looks like: It's taking forever to run, so I'll check tomorrow. |
It's working!
|
Can I supply an empty value as well? |
Sure, go ahead. I tried this and didn't work: 0dd40ce And if you can make it default, even better. |
What about adding a "none" option that in YML then gets set to |
I tried with "none" but it's passed as the literal string "none", instead of getting converted to ''. It didn't work:
So I added a condition that selects the value of the _DotNetFinalVersionKind variable depending on the value of the parameter. It worked!
|
No description provided.