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

add tracking in StaticAnalyzer of if (cfg.exists or existsAs<Type for psets #36261

Closed
slava77 opened this issue Nov 26, 2021 · 8 comments
Closed

Comments

@slava77
Copy link
Contributor

slava77 commented Nov 26, 2021

one of the pset coding requirements is to not use if (cfg.exists or existsAs<Type.

Here I'm proposing to encode this rule in the StaticAnalyzer so that the issues in the existing code can be found easily.

following https://mattermost.web.cern.ch/cms-o-and-c/pl/q9jr346jrj8xida17iaoywbqbc

@cmsbuild
Copy link
Contributor

A new Issue was created by @slava77 Slava Krutelyov.

@Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@slava77
Copy link
Contributor Author

slava77 commented Nov 26, 2021

@jpata
we talked about this earlier; this is now formalized as an issue

@makortel
Copy link
Contributor

assign core

@cmsbuild
Copy link
Contributor

New categories assigned: core

@Dr15Jones,@smuzaffar,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

@gartung
Copy link
Member

gartung commented Dec 8, 2021

#36424

@makortel
Copy link
Contributor

@makortel
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

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

No branches or pull requests

5 participants