-
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
DQM/RPCMonitorClient fixes. #4051
Conversation
A new Pull Request was created by @acimmino (Anna Cimmino) for CMSSW_7_1_X. 29 05 2014 It involves the following packages: DQM/RPCMonitorClient @ojeda, @danduggan, @rovere, @cmsbuild, @nclopezo, @deguio, @Degano can you please review it and eventually sign? Thanks. |
Please use a sensible title for the PR. It goes into the release notes. |
Dear Giulio, Should I "redo" the PR and change the title or was this just a suggestion for next time? Anna From: Giulio Eulisse [notifications@github.com] Please use a sensible title for the PR. It goes into the release notes. — |
Just edit the title (there should be an "edit" link at the top). |
PR title changes, as requested. |
I've changed the title to "DQM/RPCMonitorClient fixes." which is a bit more clean. Keep in mind that this is something people read when reading the release notes. |
+1 |
DQM/RPCMonitorClient fixes.
Bug fixes.