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

Adding consumes calls #693

Merged
merged 2 commits into from Sep 17, 2013
Merged

Adding consumes calls #693

merged 2 commits into from Sep 17, 2013

Conversation

jluetic
Copy link
Contributor

@jluetic jluetic commented Sep 2, 2013

Adding appropriate consumes calls for this package.

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 2, 2013

A new Pull Request was created by @jluetic for CMSSW_7_0_X.

Adding consumes calls

It involves the following packages:

CalibTracker/SiPixelLorentzAngle

@demattia, @rcastello can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

@nclopezo
Copy link
Contributor

nclopezo commented Sep 3, 2013

Hi,

I ran the usual tests for this pull request on top of CMSSW_7_0_X_2013-09-03-0200, all tests passed.

https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc472/366/console

you can see the artifacts here:

https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/366/

@ktf
Copy link
Contributor

ktf commented Sep 17, 2013

@rcastello @demattia any reasons why this is not signed yet?

@demattia
Copy link
Contributor

+1

ktf added a commit that referenced this pull request Sep 17, 2013
Consumes migration -- Adding consumes calls to CalibTracker/SiPixelLorentzAngle
@ktf ktf merged commit 9ff43a8 into cms-sw:CMSSW_7_0_X Sep 17, 2013
@jluetic jluetic deleted the PixelConsumesRef branch September 19, 2013 08:56
kirschen pushed a commit to kirschen/cmssw that referenced this pull request Aug 23, 2017
HeppyAnalyzer = Add met_EGCorX and met_EGCorY
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants