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

chore(bridge-ui): follow JS convention for method name #13430

Merged
merged 47 commits into from
Mar 27, 2023
Merged

Conversation

jscriptcoder
Copy link
Contributor

@jscriptcoder jscriptcoder commented Mar 23, 2023

In order to be consistent let's follow some basic conventions regarding function naming. What I'm saying here is that this is no Go lang 😉

@codecov
Copy link

codecov bot commented Mar 23, 2023

Codecov Report

Merging #13430 (6277e50) into main (cb32862) will decrease coverage by 0.79%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main   #13430      +/-   ##
==========================================
- Coverage   35.55%   34.76%   -0.79%     
==========================================
  Files         122      122              
  Lines        3741     3693      -48     
  Branches      497      497              
==========================================
- Hits         1330     1284      -46     
+ Misses       2320     2318       -2     
  Partials       91       91              
Flag Coverage Δ *Carryforward flag
bridge-ui 94.01% <100.00%> (ø)
protocol 0.00% <ø> (ø) Carriedforward from 5af628a
relayer 61.34% <ø> (-1.08%) ⬇️ Carriedforward from 5af628a
ui 100.00% <ø> (ø) Carriedforward from 5af628a

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
packages/bridge-ui/src/bridge/ERC20Bridge.ts 88.88% <100.00%> (ø)
packages/bridge-ui/src/bridge/ETHBridge.ts 88.57% <100.00%> (ø)
packages/bridge-ui/src/proof/ProofService.ts 100.00% <100.00%> (ø)
packages/bridge-ui/src/storage/StorageService.ts 95.87% <100.00%> (ø)

... and 5 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

dantaik
dantaik previously approved these changes Mar 23, 2023
dionysuzx
dionysuzx previously approved these changes Mar 23, 2023
shadab-taiko
shadab-taiko previously approved these changes Mar 24, 2023
Base automatically changed from mocking_env-vars to main March 24, 2023 07:54
@dantaik dantaik enabled auto-merge March 24, 2023 07:54
@jscriptcoder jscriptcoder self-assigned this Mar 27, 2023
@dantaik dantaik added this pull request to the merge queue Mar 27, 2023
Merged via the queue into main with commit 8ef5ce2 Mar 27, 2023
@dantaik dantaik deleted the js_method_convention branch March 27, 2023 17:01
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.

7 participants