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

build_meta: remove ansible-core Github repo links #558

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

gotmax23
Copy link
Contributor

This removes the Bug Tracker and Source Code links from the ansible package's Python metadata that point to ansible-core repository.

These links are confusing. ansible-core's issue tracker is only for issues that apply to core, not all of the ansible package. None of the source code from ansible/ansible is included directly in this package, either.

The ansible-core links are replaced with a link to the forum and a link to the ansible-build-data repository.

This only applies to ansible >= 9.0.0rc1.

This removes the Bug Tracker and Source Code links from the ansible
package's Python metadata that point to ansible-core repository.

These links are confusing. ansible-core's issue tracker is only for
issues that apply to core, not all of the ansible package. None of the
source code from ansible/ansible is included directly in this package,
either.

The ansible-core links are replaced with a link to the forum and a link
to the ansible-build-data repository.

This only applies to ansible >= 9.0.0rc1.
@gotmax23
Copy link
Contributor Author

I don't have access to ansible on Test PyPI to push a test release, so you can just imagine what this would look like there:

PyPI Screenshot

Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

I like this, though it might be better if we have more wide SC approval for this.

@ansible-community/steering-committee WDYT?

Copy link

@russoz russoz left a comment

Choose a reason for hiding this comment

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

Makes sense to me.

@mariolenz
Copy link

Sounds like a good idea.

@gotmax23
Copy link
Contributor Author

@sivel, FYI. We had some feedback on the forum that folks were unclear about where to file issues. Hopefully, this and #554 should help avoid core's issue tracker getting flooded with irrelevant issues.

@felixfontein
Copy link
Collaborator

If nobody objects until tomorrow morning (my time), I'll merge this and include this in a new release for tomorrow's Ansible 9.0.0rc1 release.

@gotmax23 gotmax23 added this to the 0.59.0 milestone Nov 14, 2023
@felixfontein felixfontein merged commit 5fee44c into ansible-community:main Nov 14, 2023
8 checks passed
@felixfontein
Copy link
Collaborator

@gotmax23 @russoz @mariolenz @Andersson007 @briantist thanks!

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.

6 participants