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
Improve Skeleton templates and scripts #29850
Comments
assign core |
New categories assigned: core @Dr15Jones,@smuzaffar,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks |
A new Issue was created by @makortel Matti Kortelainen. @Dr15Jones, @silviodonato, @dpiparo, @smuzaffar, @makortel can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
@smuzaffar The template scripts heavily assume being run in
(spurred by https://mattermost.web.cern.ch/cms-o-and-c/pl/kgpdtbtdzfyi8ezs9brdqf7awc) |
@makortel I would suggest to do something like the following in the testing script
|
Thanks @smuzaffar! I followed your suggestion in #37125, and at least in local tests it seems to work fine. |
Some follow-up items from #29840
clang-format
andclang-tidy
to ensure the generated skeletons comply with code checksAnd to not forget, #26618 has also a few items left.
The text was updated successfully, but these errors were encountered: