-
Notifications
You must be signed in to change notification settings - Fork 174
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
Remove testdata #3635
Remove testdata #3635
Conversation
"!tests/data/json/**/*", | ||
"!tests/data/postcard/**/*", |
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.
Just flagging this; I assume the tests still run without this data?
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.
This data is not used by tests. It just exists so we can look at diffs on GitHub (fingerprints and JSON contents).
in icu_datagen. If not, it's likely that japanese.rs in icu_datagen will need \ | ||
to be updated to handle the data changes. Once done, be sure to regenerate datetime/symbols@1 as well if not \ | ||
doing so already" | ||
changed in an incompatible way, or there is a new Japanese era. Run \ |
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.
Flagging this for @Manishearth
LocaleInclude::Explicit(icu_testdata::locales().into_iter().collect()); | ||
options | ||
}, | ||
Default::default(), |
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.
Question: you can do this because the default behavior is to enumerate the directories to find the locales?
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.
Yes, default behaviour is all locales.
Discussion:
LGTM: @Manishearth @sffc @robertbastian |
let out_root = | ||
std::path::Path::new(std::env!("CARGO_MANIFEST_DIR")).join("../../provider/datagen"); |
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.
Thought: Not very happy about this using a relative path, but it is in a no-publish crate, so okay
Fixes #3529