-
Notifications
You must be signed in to change notification settings - Fork 280
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
3. Travis astyle #20
3. Travis astyle #20
Conversation
Define "depends". Is it based on #16's branch, instead of master? |
I see, I guess you meant "replaces". |
I meant depends as in "is based on", but then I decided to drop the old one instead. |
Reviewed 10 of 10 files at r1. other/travis/toxcore-script, line 9 [r1] (raw file):
Just Comments from Reviewable |
Although it's out of scope of this PR, I just want to remind that there was a discussion about changing tox*_version_is_compatible() functions to use a macro, which might be an idea for a next PR someone makes. Review status: all files reviewed at latest revision, 1 unresolved discussion. Comments from Reviewable |
Review status: 9 of 14 files reviewed at latest revision, 1 unresolved discussion. other/travis/toxcore-script, line 9 [r1] (raw file):
|
It is indeed out of scope. This comment will get lost. You could instead open a ticket to keep track of the discussion. Review status: 9 of 14 files reviewed at latest revision, 1 unresolved discussion. Comments from Reviewable |
@nurupo is this one good to go now? |
0872baf
to
2d4327a
Compare
LGTM Reviewed 7 of 10 files at r1, 5 of 6 files at r2, 2 of 2 files at r3. Comments from Reviewable |
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed. Comments from Reviewable |
We run astyle on Travis and check if there is a diff. The build terminates if git finds a difference.
Reviewed 1 of 1 files at r4. Comments from Reviewable |
Depends on #16
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)