Skip to content

Conversation

@paulyuk
Copy link
Contributor

@paulyuk paulyuk commented Jul 9, 2024

Test failures can happen running tests on 1.13 -> 1.14 because test relied on implicit ToString instead of inspecting .value off of the configuration item. Also ensure print consistently handles string interpolation.

Thank you @berndverst for educating me here.

Issue reference

#1050

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • The quickstart code compiles correctly
  • You've tested new builds of the quickstart if you changed quickstart code
  • You've updated the quickstart's README if necessary
  • If you have changed the steps for a quickstart be sure that you have updated the automated validation accordingly. All of our quickstarts have annotations that allow them to be executed automatically as code. For more information see mechanical-markdown. For user guide with examples see Examples.

…on 1.13 -> 1.14

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
@paulyuk paulyuk added bug Something isn't working P1 language/python Pull requests that update Python code area/configuration labels Jul 9, 2024
@paulyuk paulyuk added this to the 1.14 milestone Jul 9, 2024
@paulyuk paulyuk requested review from berndverst and msfussell July 9, 2024 06:24
@paulyuk paulyuk self-assigned this Jul 9, 2024
@paulyuk
Copy link
Contributor Author

paulyuk commented Jul 9, 2024

@msfussell Python tests pass, ready to merge.

@paulyuk
Copy link
Contributor Author

paulyuk commented Jul 9, 2024

Merging. The JS and Tutorials failures are unrelated and have separate issues tracking.

@paulyuk
Copy link
Contributor Author

paulyuk commented Jul 9, 2024

@hhunter-ms docs-needed pls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/configuration bug Something isn't working docs-needed language/python Pull requests that update Python code P1

Projects

Development

Successfully merging this pull request may close these issues.

2 participants