-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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] Drop Conan 1.x from CCI documentation #25875
[docs] Drop Conan 1.x from CCI documentation #25875
Conversation
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
I checked all link that failed on the linter and they are fine. It's some permission limitation from the bot that's checking. |
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
* if some steps are sensitive to race conditions, monothread should be enforced. | ||
* otherwise multithreaded build should be enabled with a number of cores controlled by `tools.build:jobs` conf from host profile if it is set, otherwise to all cores of build machine. | ||
|
||
## Package Method |
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.
Package method is well covered by Conan docs already:
|
||
### Build Requirements |
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.
Tool requirements is well covered by Conan docs:
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.
Test package is not only well Covered by Conan docs, as listed in the CCI recipe templates.
docs/error_knowledge_base.md
Outdated
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.
conan center hooks is no longer running in CCI. This document does not reflect what is listed in Check results.
@@ -385,61 +357,6 @@ Keep reading in the [consuming recipes section](consuming_recipes.md#isolate-you | |||
Version ranges are currently allowed on a handful of dependencies, but not for general use. | |||
See [Dependencies Version Ranges](adding_packages/dependencies.md#version-ranges) for additional details. | |||
|
|||
## How to consume a graph of shared libraries? |
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.
It's the default in CCI
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.
mmm but possibly still needs pointing to -o "*:shared=True"
as a blanket recommendation?
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.
Addressed by 8722871
docs/labels.md
Outdated
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.
For now, the CI no uses labels to automate actions.
docs/linters.md
Outdated
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.
Previous linters were removed and current one is maturating internally only.
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.
The Checks is very clear about what is supported. This document was updated by previous CI, but with several limitations.
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
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.
Less is more 🌈
Co-authored-by: Abril Rincón Blanco <[email protected]>
Co-authored-by: Abril Rincón Blanco <[email protected]>
Co-authored-by: Abril Rincón Blanco <[email protected]>
Co-authored-by: Abril Rincón Blanco <[email protected]>
Co-authored-by: Abril Rincón Blanco <[email protected]>
Co-authored-by: Abril Rincón Blanco <[email protected]>
Summary
Update documentation to follow Conan 2.x only
Motivation
With the update of CCI to run Conan 2.x only, the documentation became outdated.
This PR remove all outdated links and of the content that's well covered by the docs.conan.io. 🧹
It's even possible to simplify even more, pointing many topics directly to Conan Docs, but I would prefer doing it in a separated PR.
Details