Skip to content

Comments

Correct log when python-version-file unspecified#429

Closed
Kurt-von-Laven wants to merge 2 commits intoactions:mainfrom
Kurt-von-Laven:patch-1
Closed

Correct log when python-version-file unspecified#429
Kurt-von-Laven wants to merge 2 commits intoactions:mainfrom
Kurt-von-Laven:patch-1

Conversation

@Kurt-von-Laven
Copy link
Contributor

@Kurt-von-Laven Kurt-von-Laven commented Jun 10, 2022

Description:
Set the versionFile variable to its default value of .python-version when it's not specified. This corrects the log statement to read: "Resolved .python-version as x.y.z" rather than "Resolved as x.y.z."

Related issue:
Fixes #428.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@dmitry-shibanov
Copy link
Contributor

Hello @Kurt-von-Laven. Thank you for your pull request. Could you please run the npm run build and npm run format commands

@Kurt-von-Laven
Copy link
Contributor Author

Thanks for the tip; done! Might be worth adding this to a contributing guide or the pull request template.

@Kurt-von-Laven
Copy link
Contributor Author

My gut feeling is that the failures are CI flakiness and unrelated to my changes, but please let me know if I am missing something.

@dmitry-shibanov
Copy link
Contributor

Hello @Kurt-von-Laven. I think your changes are not related to CI failures.

@vsafonkin
Copy link

@Kurt-von-Laven, I believe your PR #431 will resolve this issue as well, no need for this PR.

@Kurt-von-Laven
Copy link
Contributor Author

Yes, that is true. I made this PR before I was aware of the other issue. I will close this one pending the other’s acceptance.

Kurt-von-Laven and others added 2 commits June 16, 2022 12:38
Set the versionFile variable to its default value of .python-version
when it's not specified. This corrects the log statement to read:
"Resolved .python-version as x.y.z" rather than "Resolved  as x.y.z."
@Kurt-von-Laven
Copy link
Contributor Author

Superseded by #431.

@Kurt-von-Laven Kurt-von-Laven deleted the patch-1 branch June 16, 2022 19:39
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.

Log statement omits default Python version file

3 participants