Skip to content

[Qt] include and file header cleanup#4852

Merged
laanwj merged 1 commit intobitcoin:masterfrom
Diapolo:Qt_cleanup
Sep 22, 2014
Merged

[Qt] include and file header cleanup#4852
laanwj merged 1 commit intobitcoin:masterfrom
Diapolo:Qt_cleanup

Conversation

@Diapolo
Copy link
Copy Markdown

@Diapolo Diapolo commented Sep 5, 2014

  • alphabetical ordering
  • correct ordering own headers before normal headers etc.

@laanwj
Copy link
Copy Markdown
Member

laanwj commented Sep 5, 2014

Please don't submit a lot of these trivial pulls, we're swamped with pull requests at the moment.

@Diapolo
Copy link
Copy Markdown
Author

Diapolo commented Sep 16, 2014

@laanwj Ping

- alphabetical ordering
- correct ordering own headers before normal headers etc.
@BitcoinPullTester
Copy link
Copy Markdown

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4852_5e83bc404c77d6e70c62152f1f97eb264afcc80d/ for binaries and test log.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/
Contact BlueMatt on freenode if something looks broken.

@laanwj laanwj merged commit 5e83bc4 into bitcoin:master Sep 22, 2014
laanwj added a commit that referenced this pull request Sep 22, 2014
5e83bc4 [Qt] include and file header cleanup (Philip Kaufmann)
@Diapolo Diapolo deleted the Qt_cleanup branch September 22, 2014 12:20
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants