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.

Sign up for GitHub

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

Jump to bottom

Formatting issues on master #2094

Merged
merged 1 commit into from
Jun 17, 2019
Merged

Formatting issues on master #2094

merged 1 commit into from
Jun 17, 2019

Conversation

cryptocode
Copy link
Contributor

Master formatting disagrees with mine and wez's, might need formatting guards if other's get issues after this PR

@cryptocode cryptocode requested a review from wezrule June 17, 2019 13:21
@cryptocode cryptocode added this to the V19.0 milestone Jun 17, 2019
argakiig
argakiig approved these changes Jun 17, 2019
View reviewed changes
Copy link
Contributor

@argakiig argakiig left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more. 

clang-format version 6.0.1 (tags/RELEASE_601/final)

doesn't catch this either way on windows LGTM

@cryptocode cryptocode self-assigned this Jun 17, 2019
@cryptocode cryptocode merged commit 907dbf7 into nanocurrency:master Jun 17, 2019
@cryptocode cryptocode deleted the formatting/clang-format branch June 17, 2019 16:26
argakiig pushed a commit that referenced this pull request Jun 27, 2019
@cryptocode
Formatting issues on master (#2094)
bd6b178
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@argakiig argakiig argakiig approved these changes

@wezrule wezrule Awaiting requested review from wezrule

Assignees

@cryptocode cryptocode

Labels
build-error
Projects
None yet
Milestone
V19.0
Development

Successfully merging this pull request may close these issues.
2 participants
@cryptocode @argakiig