-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[12_3_X] [L1-O2O] make WriterProxyT to use ESGetToken #37690
[12_3_X] [L1-O2O] make WriterProxyT to use ESGetToken #37690
Conversation
code-checks removed the wrong EDConsumerBase inheritance from WriterProxy
the ESGetToken to the WriterProxyT implementations
Co-authored-by: Matti Kortelainen <[email protected]>
Co-authored-by: Matti Kortelainen <[email protected]>
(L1TMenu and L1TSubs need different setup - otherwise there are exceptions that cannot be handled)
…eir exceptions cannot be handled yet
A new Pull Request was created by @panoskatsoulis (Panos) for CMSSW_12_3_X. It involves the following packages:
@malbouis, @epalencia, @cmsbuild, @rekovic, @ggovi, @tvami, @cecilecaillol, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
@cmsbuild , please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-35ed15/24236/summary.html Comparison SummarySummary:
|
-db
|
yes, closing |
backport #37681