-
Notifications
You must be signed in to change notification settings - Fork 176
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
Split DataProvider into ResourceProvider and DynProvider #1554
Merged
Merged
Changes from 1 commit
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
238a44c
Initial ResourceMarker and ResourceProvider definitions
sffc 0c7f4f2
Checkpoint
sffc b60b909
Remove redundant module annotation
sffc a9e1b8d
Big refactor around new traits
sffc 10dbe6f
cargo check --all-targets --all-features in core crate
sffc 600cfbf
icu_provider_blob building
sffc d2ce258
icu_provider_fs building
sffc 3b0b5c9
Delete blanket impls
sffc e591f9a
Add DataProvider::cast method
sffc aaa8132
icu_properties via DynProvider
sffc 9a19c8c
Fixing up some components
sffc 9098d3e
Merge remote-tracking branch 'upstream/main' into resourcemarker
sffc c1af44d
More migrations
sffc 70b6821
More migrations
sffc 86b1e91
ResourceProvider in plurals
sffc f7412eb
Split plural rules constructors
sffc f330a8a
Migrate call sites to type-specific PluralRules constructors
sffc 579179b
fmt
sffc beaca9c
Add basic tests to icu_provider_macros
sffc 4174efd
Add basic ResourceMarker to macro
sffc e1ecce8
Improve test
sffc 54a6a94
Add multi-key support
sffc 7077b75
Make docs more consistent
sffc d72d419
Add docs example
sffc f858b38
Add namespace to ZeroCopyFrom in yoke derive
sffc 1e6b178
Use new data_struct attribute across components
sffc d97f077
Clean up imports
sffc c885e29
uprops building
sffc 1418c81
Migrating cldr
sffc c0e3444
ICU4X cargo check --all-features
sffc 91e0f5e
Fixing some tests
sffc f36c233
ICU4X cargo check --all-targets --all-features
sffc 5a3374f
Merge remote-tracking branch 'upstream/main' into resourcemarker
sffc 8a40399
fmt
sffc edd600e
cargo quick
sffc fda78d1
Work around Rust bug rust-lang/rust#93470
sffc 87fb440
Merge remote-tracking branch 'upstream/main' into resourcemarker
sffc c0262c1
Fix resource_path_to_string
sffc c112f5b
Fixing docs tests in icu_provider
sffc 2f34269
Fix failing bench test
sffc a08a744
More docs tests fixes
sffc bb1479a
Another docs test down
sffc 4010e7a
Re-write icu_provider crate-level docs
sffc 7150659
Doc link fixes
sffc 285cccf
diplomat regen
sffc 7217ea7
A few more docs updates + ForkByKey for DynProvider
sffc 0d93723
icu_provider links fixed
sffc 0f010c0
Fixing remaining docs links
sffc 68b45cd
list_formatter fixups
sffc e04ba20
clippy & tidy
sffc 6e4bac1
Docs for DynProvider and ResourceProvider
sffc 33498b9
Fix plural rules FFI test
sffc bdce762
Fix FFI test take 2
sffc 860e21f
Merge remote-tracking branch 'upstream/main' into resourcemarker
sffc 9772515
Fix build
sffc 842215a
Remove unneeded type info
sffc 127aaf0
Fix another compile error
sffc 099ab34
try_langid -> get_langid
sffc b2c8f7d
Update data_struct macro to require explicit marker symbol path
sffc 8a47ffa
Migrate call sites to new macro
sffc d2902af
Make PluralRulesV1Marker public
sffc 14fefcc
Remove ResourcePath and DataRequestOld
sffc 51cf962
Remove unused file
sffc 8f992bf
Update provider/macros/src/lib.rs
sffc 1217eed
Merge branch 'main' into resourcemarker
sffc 16120c6
fmt
sffc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
issue: I'm not a huge fan of "type 1" and "type 2" naming, can we just call these any providers and buffered providers?
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.
#1576 (comment)