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
fix: bump legacy-table-chart to 0.11.20 #9286
Conversation
017baea
to
69ab995
Compare
69ab995
to
7969905
Compare
7969905
to
8c4902e
Compare
Why are there so many package lock changes here? |
Did a fresh rm -rf node_modules package-lock.json && ENV="" NODE_ENV="development" npm install Maybe not a good idea? |
8c4902e
to
b3528e0
Compare
@ktmud That is kinda dangerous. |
@@ -7670,28 +7670,28 @@ | |||
"dependencies": { | |||
"abbrev": { | |||
"version": "1.1.1", | |||
"resolved": "", | |||
"resolved": false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Empty strings become false
s again. 🤦♂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol this i can deal with :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks!
@@ -7670,28 +7670,28 @@ | |||
"dependencies": { | |||
"abbrev": { | |||
"version": "1.1.1", | |||
"resolved": "", | |||
"resolved": false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol this i can deal with :P
CATEGORY
SUMMARY
Minor fixes for the data table:
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before
After
TEST PLAN
ADDITIONAL INFORMATION
N/A
REVIEWERS
@etr2460 @graceguo-supercat @kristw