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

Switch pbr setup.cfg metadata to setuptools metadata #535

Merged
merged 1 commit into from
May 16, 2023

Conversation

sivel
Copy link
Member

@sivel sivel commented May 16, 2023

pbr used some differing metadata keys from what setuptools uses/documents.

Diff of the `METADATA` files included in the wheel...

--- metadata-before.txt	2023-05-16 08:55:22
+++ metadata-after.txt	2023-05-16 08:57:43
@@ -22,9 +22,46 @@
 Classifier: Topic :: System :: Systems Administration
 Classifier: Topic :: Utilities
 Requires-Python: >=3.9
+Description-Content-Type: text/markdown
 License-File: LICENSE.md
 Requires-Dist: PyYAML
 Requires-Dist: requirements-parser
 Requires-Dist: bindep
 Requires-Dist: jsonschema
 
+[![CI](https://github.com/ansible/ansible-builder/actions/workflows/ci.yml/badge.svg?branch=devel)](https://github.com/ansible/ansible-builder/actions?query=branch%3Adevel)
+[![codecov](https://codecov.io/gh/ansible/ansible-builder/branch/devel/graph/badge.svg?token=4F6P3DBI40)](https://codecov.io/gh/ansible/ansible-builder)
+
+# Ansible Builder
+
+Ansible Builder is a tool that automates the process of building execution
+environments using the schemas and tooling defined in various Ansible
+Collections and by the user.
+
+See the readthedocs page for `ansible-builder` at:
+
+https://ansible-builder.readthedocs.io/en/stable/
+
+
+## Get Involved:
+
+* We use [GitHub issues](https://github.com/ansible/ansible-builder/issues) to
+  track bug reports and feature ideas
+* Want to contribute, check out our [guide](CONTRIBUTING.md)
+* Join us in the `#ansible-builder` channel on Libera.chat IRC
+* For the full list of Ansible email Lists, IRC channels and working groups,
+  check out the [Ansible Mailing
+  lists](https://docs.ansible.com/ansible/latest/community/communication.html#mailing-list-information)
+  page of the official Ansible documentation.
+
+## Code of Conduct
+
+We ask all of our community members and contributors to adhere to the [Ansible
+code of
+conduct](http://docs.ansible.com/ansible/latest/community/code_of_conduct.html). If
+you have questions, or need assistance, please reach out to our community team
+at [codeofconduct@ansible.com](mailto:codeofconduct@ansible.com)
+
+## License
+
+[Apache License v2.0](./LICENSE.md)

Technically summary and home_page didn't need changes, but I switched them to adhere to the setup.cfg docs at https://setuptools.pypa.io/en/latest/userguide/declarative_config.html

Fixes #532

@sivel sivel requested a review from a team as a code owner May 16, 2023 13:59
@github-actions github-actions bot added the needs_triage New item that needs to be triaged label May 16, 2023
Copy link
Contributor

@Shrews Shrews left a comment

Choose a reason for hiding this comment

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

@Shrews Shrews changed the title Switch pbr setup.cfg metadata to setuptools metadata. Fixes #532 Switch pbr setup.cfg metadata to setuptools metadata May 16, 2023
@nitzmahone nitzmahone merged commit d414c72 into ansible:devel May 16, 2023
@nitzmahone nitzmahone removed the needs_triage New item that needs to be triaged label May 16, 2023
Shrews pushed a commit to Shrews/ansible-builder that referenced this pull request Jun 8, 2023
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.

setuptools_scm switch broke PyPI metadata
4 participants