Skip to content

change tree variable name#16

Merged
gpetruc merged 5 commits into
cms-analysis:74Xfrom
perrozzi:final_all_variables
Nov 6, 2015
Merged

change tree variable name#16
gpetruc merged 5 commits into
cms-analysis:74Xfrom
perrozzi:final_all_variables

Conversation

@perrozzi
Copy link
Copy Markdown
Contributor

No description provided.

@perrozzi
Copy link
Copy Markdown
Contributor Author

@gpetruc could you please merge or comment ASAP? @GLP90 needs to start the production

@gpetruc
Copy link
Copy Markdown
Contributor

gpetruc commented Oct 20, 2015

Hi Luca,

GLP does not need the PR to be merged in order to start the production.

Giovanni
Il 20-ott-2015 01:07 PM, "perrozzi" notifications@github.com ha scritto:

@gpetruc https://github.com/gpetruc could you please merge or comment
ASAP? @GLP90 https://github.com/GLP90 needs to start the production


Reply to this email directly or view it on GitHub
#16 (comment)
.

@perrozzi
Copy link
Copy Markdown
Contributor Author

not to merge, but the "green light" from somebody having an independent look at the code yes. not necessarily you, I agree and apologize for putting pressure to you.
but if not you somebody else should reply to the PR and comment to avoid problems...

ciao
luca

@HuguesBrun HuguesBrun mentioned this pull request Nov 6, 2015
@gpetruc gpetruc merged commit f8d4118 into cms-analysis:74X Nov 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants