-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Modified functional scope to block scope in javascript #11526
Conversation
This pull request generated screenshots of many common pages in the running app. You should be able to download and view them here: |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #11526 +/- ##
===========================================
+ Coverage 50.26% 83.21% +32.94%
===========================================
Files 93 93
Lines 5946 5772 -174
===========================================
+ Hits 2989 4803 +1814
+ Misses 2957 969 -1988
|
Hi @Josephsanya, can you add a link to the issue?
|
Hello @TildaDares , I didn't fix it from an issue. It was just some improvements I made while going through the code. |
Hi @Josephsanya, we prefer to create issues first so that other contributors don't try to solve the same issues. It also gives the maintainers time to review the issue before you open a PR for it. |
Code Climate has analyzed commit 78a8076 and detected 0 issues on this pull request. View more on Code Climate. |
rake test
@publiclab/reviewers
for help, in a comment below