Skip to content
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

Check if it makes sense to guard against mistypes in certain properties #16042

Closed
AbrilRBS opened this issue Apr 8, 2024 · 2 comments · Fixed by #16848
Closed

Check if it makes sense to guard against mistypes in certain properties #16042

AbrilRBS opened this issue Apr 8, 2024 · 2 comments · Fixed by #16848
Assignees

Comments

@AbrilRBS
Copy link
Member

AbrilRBS commented Apr 8, 2024

          @valgur That's the second time I've been bitten by that exact problem, yes a check for that would be wonderful. [..]

Originally posted by @Ahajha in conan-io/conan-center-index#22599 (comment)

if this seems like something that some people are falling for, maybe it's worth a check to whether we can help there, either a warning/error o silently fixing it

@Ahajha
Copy link

Ahajha commented Apr 8, 2024

FWIW, searching through CCI I don't see this happening anywhere else, so either it was just a blunder on my part twice or the reviews have been good enough to catch it. That said, I still think it worth checking for, since currently the (cryptic) error will occur at package consumption time which is not ideal.

@memsharded
Copy link
Member

Let's add check_type= argument to .get_property(), similar to conf system

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants