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

refactor: move superset-ui/dimension to core #732

Merged
merged 1 commit into from
Aug 26, 2020

Conversation

ktmud
Copy link
Contributor

@ktmud ktmud commented Aug 11, 2020

🏠 Internal

Keep merging more superset-ui packages.

Related: #728 #729 #730

@vercel
Copy link

vercel bot commented Aug 11, 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/32dq94ir8
✅ Preview: https://superset-ui-git-fork-ktmud-monopackage-move-dimension.superset.vercel.app

@codecov
Copy link

codecov bot commented Aug 20, 2020

Codecov Report

Merging #732 into monopackage will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           monopackage     #732   +/-   ##
============================================
  Coverage        24.29%   24.29%           
============================================
  Files              337      337           
  Lines             7697     7697           
  Branches           955      955           
============================================
  Hits              1870     1870           
  Misses            5753     5753           
  Partials            74       74           
Impacted Files Coverage Δ
...es/superset-ui-chart/src/components/SuperChart.tsx 100.00% <ø> (ø)
...perset-ui-core/src/dimension/computeMaxFontSize.ts 100.00% <ø> (ø)
...ui-core/src/dimension/getMultipleTextDimensions.ts 100.00% <ø> (ø)
...superset-ui-core/src/dimension/getTextDimension.ts 100.00% <ø> (ø)
...ages/superset-ui-core/src/dimension/mergeMargin.ts 100.00% <ø> (ø)
...ages/superset-ui-core/src/dimension/parseLength.ts 100.00% <ø> (ø)
.../superset-ui-core/src/dimension/svg/LazyFactory.ts 100.00% <ø> (ø)
...es/superset-ui-core/src/dimension/svg/constants.ts 100.00% <ø> (ø)
...t-ui-core/src/dimension/svg/createHiddenSvgNode.ts 100.00% <ø> (ø)
...perset-ui-core/src/dimension/svg/createTextNode.ts 100.00% <ø> (ø)
... and 13 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 fdb96fb...d993aaf. Read the comment docs.

@ktmud ktmud marked this pull request as ready for review August 20, 2020 22:24
@ktmud ktmud requested a review from a team as a code owner August 20, 2020 22:24
@@ -29,9 +29,6 @@
},
{
"path": "../../packages/superset-ui-core"
},
{
"path": "../../packages/superset-ui-dimension"
}
]
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This file is autogenerated by nimbus so I wouldn't worry much about the missing EOF.


if [ ! "$tscExitCode" ]; then
echo "compiled in ${duration}"
fi
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Needed to give beautiful feedback like this:

image

Copy link
Member

Choose a reason for hiding this comment

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

🌈 ❗

@ktmud ktmud force-pushed the monopackage-move-dimension branch from 01824f2 to d993aaf Compare August 26, 2020 03:43
@ktmud ktmud merged commit 4f25e7c into apache-superset:monopackage 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)
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)
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants