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

Some docs cleanups #5825

Merged
merged 1 commit into from
Aug 7, 2023
Merged

Some docs cleanups #5825

merged 1 commit into from
Aug 7, 2023

Conversation

jeffwidman
Copy link
Contributor

  1. pypi.python.org now redirects to pypi.org, so let's point folks towards that to save them the redirect.
  2. Travis CI has struggled mightily ever since they were acquired by a PE firm. Most open source projects have moved away from them.
  3. Update the python version in the example so it doesn't look quite so dated.
  4. Use the name key for sources as that's now required.

I didn't include a news fragment as I thought the changes were too trivial to list.

I also didn't take the time to regenerate the lockfiles after changing the Pipfile... that means the hashes are going to be wrong, but for the purposes of docs I think that's fine... no human will notice, and if they do run pipenv against the example and are surprised by the outcome they're welcome to submit a PR fixing that too. But for now I think tis fine. Certainly this PR leaves things better then they were.

1. `pypi.python.org` now redirects to `pypi.org`, so let's point folks towards that to save them the redirect.
2. Travis CI has struggled mightily ever since they were acquired by a PE firm. Most open source projects have moved away from them.
3. Update the python version in the example so it doesn't look quite so dated.
4. Use the `name` key for `sources` as that's now [required](#5370 (comment)).
@matteius matteius merged commit 549c661 into pypa:main Aug 7, 2023
@matteius
Copy link
Member

matteius commented Aug 7, 2023

Thanks Jeff!

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.

2 participants