-
Notifications
You must be signed in to change notification settings - Fork 46
[conan center] check self.requires.add (and self.build_requires.add) #175
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
Comments
current usage:
|
@jgsogo should we introduce a default for it in 1.24.1? |
We should fix
No, there is no need for a default after almost a month. There has been only one issue related to this change (conan-io/conan#6667). Thanks @SSE4 for finding all the usages 🙌 |
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
#175 Do not allow requires.add (KB-H044)
original conversation: conan-io/conan-center-index#1293 (comment)
some recipes (may) contain
self.requires.add
orself.build_requires.add
.these methods aren't currently documented.
they were documented in the past and used across recipes in previous versions of Conan center, but right now they are not part of public API.
/cc @uilianries @jgsogo
The text was updated successfully, but these errors were encountered: