Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Version specification in main text #10

Conversation

sneakers-the-rat
Copy link

Currently version specifiers are just in an admonition, and the examples all use dependency specifiers without versions. I think it's important to model version specification in the tutorials, since they probably won't see many real packages with wide open version specifiers in the wild, and they are the things that are the most troublesome/require the most maintenance over time/will be the source of the most bugs if left without version spec.

Currently

Screenshot 2024-03-01 at 2 59 37 PM Screenshot 2024-03-01 at 2 59 47 PM

This PR

After the unchanged start of the section...

Screenshot 2024-03-01 at 2 51 10 PM Screenshot 2024-03-01 at 2 51 19 PM

So now the "pin dependencies with caution" is just about pinning and narrow version constraints, and it also has a specific example for poetry, and the main text has a discussion about version specification.

I also changed the dependencies array throughout to match.

Something that newbies will also experience (i mean i am currently dealing with a problem like this) some cryptic messages about version incompatibility from requires-python saying they need to add an upper bound. I added a quick note there saying that lower bounds are just one way to specify it, but that you can use the more general version spec syntax there.

IDK why gh's syntax highlighter is showing everything as red...

@sneakers-the-rat
Copy link
Author

pyOpenSci#191

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants