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

Correct log when python-version-file unspecified #429

Closed
wants to merge 2 commits into from

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