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

Revert "[94X] L1 no hack conditions - cleaned" #20921

Merged

Conversation

davidlange6
Copy link
Contributor

Reverts #20871. Seems to be breaking WF 145. First we'll confirm that.

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/PR-20921/1456

Code check has found code style and quality issues which could be resolved by applying a patch in https://cmssdt.cern.ch/SDT/code-checks/PR-20921/1456/git-diff.patch
e.g. curl https://cmssdt.cern.ch/SDT/code-checks/PR-20921/1456/git-diff.patch | patch -p1

You can run scram build code-checks to apply code checks directly

@davidlange6
Copy link
Contributor Author

ok, well just merge and see if things are fixed.

@davidlange6
Copy link
Contributor Author

please test

@davidlange6
Copy link
Contributor Author

merge

@davidlange6 davidlange6 merged commit 8c0f84b into master Oct 15, 2017
@smuzaffar smuzaffar deleted the revert-20871-pr94x-l1-noHackConditions-fromPR20451-cleaned branch November 4, 2017 08:17
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

2 participants