-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Support other data types for DataViewer #5093
Conversation
Beginning of functional test
export const noRowsInDataViewer = localize('DataScience.noRowsInDataViewer', 'Fetching data ...'); | ||
export const pandasTooOldForViewingFormat = localize('DataScience.pandasTooOldForViewingFormat', 'Python package \'pandas\' is version {0}. Version 0.20 or greater is required for viewing data.'); | ||
export const pandasRequiredForViewing = localize('DataScience.pandasRequiredForViewing', 'Python package \'pandas\' is required for viewing data.'); | ||
export const valuesColumn = localize('DataScience.valuesColumn', 'values'); |
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.
missing in the nls.json
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.
Or actually looks like you are just using as a constant. Just a constant then?
In reply to: 271822249 [](ancestors = 271822249)
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.
For #4677
Add code for other data types
Add new functional tests for those data types
package-lock.json
has been regenerated by runningnpm install
(if dependencies have changed)