Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

refactor: move superset-ui/validator to core #757

Merged

Conversation

ktmud
Copy link
Contributor

@ktmud ktmud commented Aug 26, 2020

🏠 Internal

Related: #728 #729 #730 #732 #755 #756

@ktmud ktmud requested a review from a team as a code owner August 26, 2020 17:33
@vercel
Copy link

vercel bot commented Aug 26, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/superset/superset-ui/evsjg9spl
✅ Preview: https://superset-ui-git-fork-ktmud-monopackage-move-validator.superset.vercel.app

@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #757 into monopackage-move-style will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                   Coverage Diff                   @@
##           monopackage-move-style     #757   +/-   ##
=======================================================
  Coverage                   24.29%   24.30%           
=======================================================
  Files                         337      338    +1     
  Lines                        7697     7698    +1     
  Branches                      956      956           
=======================================================
+ Hits                         1870     1871    +1     
  Misses                       5753     5753           
  Partials                       74       74           
Impacted Files Coverage Δ
...et-ui-chart-controls/src/shared-controls/index.tsx 40.44% <ø> (ø)
...set-ui-core/src/validator/legacyValidateInteger.ts 100.00% <ø> (ø)
...rset-ui-core/src/validator/legacyValidateNumber.ts 100.00% <ø> (ø)
.../superset-ui-core/src/validator/validateInteger.ts 100.00% <ø> (ø)
...superset-ui-core/src/validator/validateNonEmpty.ts 100.00% <ø> (ø)
...s/superset-ui-core/src/validator/validateNumber.ts 100.00% <ø> (ø)
...egacy-plugin-chart-event-flow/src/controlPanel.jsx 0.00% <ø> (ø)
...ns/legacy-plugin-chart-heatmap/src/controlPanel.js 0.00% <ø> (ø)
.../legacy-plugin-chart-histogram/src/controlPanel.js 0.00% <ø> (ø)
...legacy-plugin-chart-partition/src/controlPanel.jsx 0.00% <ø> (ø)
... and 6 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 caa6202...691ddab. Read the comment docs.

@ktmud ktmud merged commit bed0aed into apache-superset:monopackage-move-style Aug 26, 2020
ktmud added a commit that referenced this pull request Aug 26, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
ktmud added a commit that referenced this pull request Aug 26, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
ktmud added a commit that referenced this pull request Sep 2, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
8: move superset-ui/chart-composition to core (#759)
ktmud added a commit that referenced this pull request Sep 2, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
8: move superset-ui/chart-composition to core (#759)
ktmud added a commit that referenced this pull request Sep 3, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
8: move superset-ui/chart-composition to core (#759)
9: move superset-ui/chart to core (#760)
ktmud added a commit that referenced this pull request Sep 3, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
8. move superset-ui/chart-composition to core (#759)
9. move superset-ui/chart to core (#760)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants