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

Applying prettier to all files #32

Merged
5 commits merged into from
Jul 20, 2022
Merged

Applying prettier to all files #32

5 commits merged into from
Jul 20, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jun 30, 2022

Scope

Run prettier on all changes, not just those under bridge/

Why?

More standardized code formatting.

@ghost ghost requested a review from vtleonardo July 6, 2022 14:21
@ghost
Copy link
Author

ghost commented Jul 6, 2022

@vtleonardo This should cover a few files from your PR.

@github-actions github-actions bot added actions dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code proto docker area/docs labels Jul 6, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 6, 2022

Codecov Report

Merging #32 (0f098c4) into main (bb2e65f) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   45.61%   45.62%   +0.01%     
==========================================
  Files         214      214              
  Lines       38504    38504              
==========================================
+ Hits        17563    17569       +6     
+ Misses      19007    19005       -2     
+ Partials     1934     1930       -4     
Impacted Files Coverage Δ
peering/active.go 83.96% <0.00%> (+0.94%) ⬆️
badgerTrie/smt_merkle_proof.go 64.84% <0.00%> (+3.90%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb2e65f...0f098c4. Read the comment docs.

@ghost ghost merged commit 61cf228 into main Jul 20, 2022
@ghost ghost deleted the prettier-all-the-things branch July 20, 2022 00:37
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
actions area/docs dependencies Pull requests that update a dependency file docker javascript Pull requests that update Javascript code proto
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant