Skip to content

Fix docstrings from Conan API and some tools #10983

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

Merged
merged 5 commits into from
Apr 12, 2022

Conversation

czoido
Copy link
Contributor

@czoido czoido commented Apr 6, 2022

Some fixes for docstrings in the Conan API and tools, now that we are using autodoc, the format should be like this: https://sphinx-rtd-tutorial.readthedocs.io/en/latest/docstrings.html

This will fix: conan-io/docs#2492

@czoido czoido added this to the 2.0.0-alpha7 milestone Apr 6, 2022
@czoido czoido marked this pull request as draft April 6, 2022 14:06
@czoido czoido marked this pull request as ready for review April 6, 2022 14:41
@czoido czoido changed the title Fix docstrings from Conan API Fix docstrings from Conan API and some tools Apr 6, 2022
Copy link
Member

@memsharded memsharded left a comment

Choose a reason for hiding this comment

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

Lets make sure that the API docs have clear (manual) red note saying it is experimental and subject to change. We will not freeze the Python API in 2.0 GA, we will wait a couple of releases into 2.X to declare the Python API stable. It needs real usage to be stabilized.

@czoido
Copy link
Contributor Author

czoido commented Apr 7, 2022

Lets make sure that the API docs have clear (manual) red note saying it is experimental and subject to change. We will not freeze the Python API in 2.0 GA, we will wait a couple of releases into 2.X to declare the Python API stable. It needs real usage to be stabilized.

I have added a api_warning.rst to this PR: https://github.com/conan-io/docs/pull/2492/files

@lasote lasote merged commit d90b205 into conan-io:develop2 Apr 12, 2022
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.

3 participants