Skip to content

Updated settings.html #13299

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

Closed
wants to merge 1 commit into from
Closed

Updated settings.html #13299

wants to merge 1 commit into from

Conversation

sabre-code
Copy link
Contributor

Based on #13143 . Code initially pointed to setup.py. Now it points to pyproject.toml. Just want to know whether in the same line setup() should be changed with something else? If current change is fine please merge.

Based on pypi#13143 . Code initially pointed to setup.py. Now it points to pyproject.toml
@sabre-code sabre-code requested a review from a team as a code owner March 27, 2023 07:49
@miketheman
Copy link
Member

Thanks @sabre-code ! Did you see the open question on the original issue that was not addressed?
#11985 (comment)

@sabre-code
Copy link
Contributor Author

Hello @miketheman. If you are hinting about the the link update issue, yes I have gone through it. Can you help me with what exactly should be updated as it's mentioned "change the name". And the part of updating the link to the one given in comment is clear to me. Just clarify the "name" thing. I will get it done. Thanks

@@ -102,7 +102,7 @@ <h3>{% trans %}<abbr title="two factor authentication">2FA</abbr> requirement{%
<h3>{% trans %}Project description and sidebar{% endtrans %}</h3>
<p>
{% trans project_name=project.name, setup_args_href='https://packaging.python.org/tutorials/distributing-packages/#setup-args', twine_docs_href='https://twine.readthedocs.io/en/latest/', distribution_href='https://packaging.python.org/tutorials/distributing-packages/' %}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On this line setup_args_href refers to setup.py arguments, which is then used in the content line.
Rename to something like pyproject_args_href and update the target URL to a correct anchor in the documentation, and then use the new name in line 105 <a href= tag

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#13310 Hello please check this. Done. If anything else happy to work it out.

@sabre-code sabre-code deleted the patch-1 branch March 27, 2023 18:28
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