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: apply capitalization guidelines - iteration 5 (#12343) #12451

Merged

Conversation

michael-s-molina
Copy link
Member

SUMMARY

Apply capitalization guidelines defined in #12343.

@junlincc

TEST PLAN

Check if modified texts are following capitalization guidelines.

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

@junlincc
Copy link
Member

please specify all your changes in this PR, include screenshot and test plan, also for all your future PRs. @michael-s-molina 🙏

@codecov-io
Copy link

codecov-io commented Jan 12, 2021

Codecov Report

Merging #12451 (f2f012f) into master (5504762) will increase coverage by 2.10%.
The diff coverage is 25.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #12451      +/-   ##
==========================================
+ Coverage   64.55%   66.65%   +2.10%     
==========================================
  Files        1018     1018              
  Lines       49787    49793       +6     
  Branches     4971     4877      -94     
==========================================
+ Hits        32141    33192    +1051     
+ Misses      17468    16478     -990     
+ Partials      178      123      -55     
Flag Coverage Δ
cypress 50.95% <25.00%> (+13.75%) ⬆️
javascript 60.94% <25.00%> (ø)
python 63.67% <ø> (-0.46%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...perset-frontend/src/datasource/DatasourceModal.tsx 73.77% <ø> (ø)
...nd/src/explore/components/ExploreViewContainer.jsx 80.00% <ø> (+5.16%) ⬆️
...rontend/src/explore/components/PropertiesModal.tsx 61.11% <ø> (ø)
...rset-frontend/src/explore/components/SaveModal.tsx 91.01% <ø> (+4.49%) ⬆️
...rc/explore/components/controls/AnnotationLayer.jsx 51.17% <0.00%> (+48.82%) ⬆️
...ore/components/controls/AnnotationLayerControl.jsx 81.42% <ø> (+32.85%) ⬆️
.../explore/components/controls/DatasourceControl.jsx 62.71% <ø> (ø)
...Control/AdhocFilterEditPopoverSimpleTabContent.jsx 78.98% <ø> (ø)
...terControl/AdhocFilterEditPopoverSqlTabContent.jsx 65.51% <ø> (ø)
...rols/MetricControl/AdhocMetricEditPopoverTitle.jsx 91.17% <100.00%> (ø)
... and 138 more

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 5504762...f2f012f. Read the comment docs.

@adam-stasiak
Copy link
Contributor

looks good to me

@mihir174
Copy link
Contributor

Other than "Open Datasource tab" all looks good! Thanks for doing this 🙏

Copy link
Member

@rusackas rusackas left a comment

Choose a reason for hiding this comment

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

LGTM, but holding off from merging.

@michael-s-molina michael-s-molina force-pushed the apply-capitalization-guidelines-5 branch 2 times, most recently from c8c7f29 to 25d428b Compare January 13, 2021 14:10
@rusackas rusackas merged commit e80cd61 into apache:master Jan 22, 2021
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.

None yet

6 participants