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
docs for full_transitive_package_id #1575
docs for full_transitive_package_id #1575
Conversation
I'm asking @czoido to review it too. It is needed that someone that hasn't been involved in the PRs understands this piece of docs |
@memsharded , have a look at the review and merge @czoido , please, review the docs, how a newcomer to Conan will read about this (with/without this flag activated)? |
Added a infographic and improved the text. I think it should be more understandable now. |
Co-Authored-By: Daniel <danimanzaneque@gmail.com>
Co-Authored-By: Carlos Zoido <mrgalleta@gmail.com>
- ``App/1.0`` depends on ``PkgC/1.0`` and ``PkgA/1.0`` | ||
- ``PkgC/1.0`` depends only on ``PkgB/1.0`` | ||
- ``PkgB/1.0`` depends on ``PkgA/1.0``, and defines ``self.info.header_only()`` in its ``package_id()`` | ||
- We are using ``full_version_mode`` |
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 move this to the line 722: "Lets say that we have the following scenario where we are using full_version_mode:"
In my humble opinion, if the point of the section is to explain the propagation of the packageID and what |
Docs for conan-io/conan#6451