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

Improve wording and "fix example" (remove 3.13) on testing against pre-release #979

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

Conversation

yarikoptic
Copy link

Description:

3.13 was added in a mass tune up in 0b93645 , without adjusting the wording. With 3.13 listed there too, example does not really make much sense. So I decided to make it explicit in wording and remove 3.13, so whenever next refactoring to add 3.14 to be added to every line where 3.13 is -- this would not even come to attention

Check list:

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

…e-releases

3.13 was added in a mass tune up in 0b93645 , without
adjusting the wording. With 3.13 listed there too, example does not really make much
sense. So I decided to make it explicit in wording and remove 3.13, so whenever next
refactoring to add 3.14 to be added to every line where 3.13 is -- this would not
even come to attention
@yarikoptic yarikoptic requested a review from a team as a code owner December 5, 2024 21:14
@GIgako19929
Copy link

This pull request includes updates to the documentation to clarify the behavior of the allow-prereleases flag and to correct an example workflow.

Documentation improvements:

  • docs/advanced-usage.md: Updated the description to clarify that the allow-prereleases flag defaults to false and provided a more accurate historical reference for when Python 3.12 was not generally available.
  • docs/advanced-usage.md: Corrected the example workflow by removing the unsupported Python 3.13 version from the matrix configuration.

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.

3 participants