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

Add .editorconfig #33

Merged
merged 1 commit into from
Aug 22, 2018
Merged

Add .editorconfig #33

merged 1 commit into from
Aug 22, 2018

Conversation

debris
Copy link
Contributor

@debris debris commented Aug 22, 2018

We need it as we use non-standard formatting and indent

#29 (comment)

@ordian
Copy link
Member

ordian commented Aug 22, 2018

Related question: are we planning on using rustfmt eventually? This would be helpful, especially for external contributors.

@NikVolf NikVolf merged commit 13cae43 into master Aug 22, 2018
@NikVolf NikVolf deleted the editorconfig branch August 22, 2018 15:56
sorpaas pushed a commit that referenced this pull request Jan 24, 2019
[WIP] Test basic algebraic laws with quickcheck
sorpaas pushed a commit that referenced this pull request Jan 24, 2019
* Prefix `Ox` for lower hex for alternate flag

Should fix openethereum/parity-ethereum#8393
As per https://doc.rust-lang.org/std/fmt/trait.LowerHex.html

* Use hex alternate format for debug
tomaka pushed a commit to tomaka/parity-common that referenced this pull request Apr 29, 2019
* Prefix `Ox` for lower hex for alternate flag

Should fix openethereum/parity-ethereum#8393
As per https://doc.rust-lang.org/std/fmt/trait.LowerHex.html

* Use hex alternate format for debug
tomaka pushed a commit to tomaka/parity-common that referenced this pull request Apr 29, 2019
* Prefix `Ox` for lower hex for alternate flag

Should fix openethereum/parity-ethereum#8393
As per https://doc.rust-lang.org/std/fmt/trait.LowerHex.html

* Use hex alternate format for debug
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.

4 participants