-
Notifications
You must be signed in to change notification settings - Fork 674
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
Feature/#507 separate package long desc with readme #1284
Feature/#507 separate package long desc with readme #1284
Conversation
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would take the opportunity to:
- Remove all sub-package
CODE_OF_CONDUCT.md
- Merge all sub-package
INSTALLATION.md
intoREADME.md
- Remove for all sub-packages, the 3 sections (running the tests, contributing, and code of conduct) of the
package_desc.md
Co-authored-by: Jean-Robin <jeanrobin.medori@avaiga.com>
Since each sub taipy folder is still a standalone package, I think it's a good practice to have its own "CODE_OF_CONDUCT.md", even if it's duplicated, we never change it anyway. For the "INSTALLATION.md", I think we can keep it but no strong opinion. Keep in mind that the content of the installation instruction can be added to the "package_desc.md" as a link to the "INSTALLATION.md" file, it must be a Github link, which is possible. I would keep the contributing and code of conduct sections in Pypi description since it's quite useful information and link directly to our doc anyway |
package_desc.md
Outdated
## Contributing | ||
|
||
Writing code isn’t the only way to contribute to Taipy. You can also: | ||
- help us stay on top of new and old issues | ||
- develop tutorials, presentations, and other educational materials | ||
- maintain and improve [our website](https://www.taipy.io) and [documentation](https://docs.taipy.io/en/latest/) | ||
- review pull requests | ||
- help with outreach and onboard new contributors | ||
|
||
For more information about the ways you can contribute to Taipy, visit our | ||
[Contributing Guide](https://docs.taipy.io/en/latest/contributing/contributing/). | ||
If you’re unsure where to start or how your skills fit in, reach out on Discord. You can also ask on | ||
GitHub, by opening a new issue or leaving a comment on a relevant issue that is already open. | ||
|
||
If you are new to contributing to open source projects, [this guide](https://opensource.guide/how-to-contribute/) | ||
helps explain why, what, and how to successfully get involved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree; I think Pypi users want a description, information on license, security and maintenance, and a link to the documentation. Since you can't contribute on Taipy, a contribution section doesn't make much sense.
Having just one "CODE_OF_CONDUCT.md" is enough, IMHO, because, as you said, it's the same everywhere, and it is not helpful information on Pypi. |
… the package long description
Resolves #507
In this PR: