-
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
Revert datagen fs feature #3359
Conversation
…e-org#3354)" This reverts commit 98a47f3.
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.
I have a mild preference for #3357
Either way, go ahead and yank 1.2.2. No need to wait for that |
Also add #3355 to the changelog before you release anything. |
Let's discuss this a bit in the meeting |
yanked, yoinked, and yeeted |
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 with follow-up in #3365
Reverts #3354, we don't actually need it that much (it doesn't have an effect on
icu_provider
"sync"
) and we landed it in a way that was accidentally breakingThis PR will also involve yanking
icu_datagen
1.2.2Alternative to #3357