Skip to content
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

jquery.i18n.fallbacks.js: Add sr-cyrl as fallback for sr #243

Merged
merged 1 commit into from
Aug 18, 2022

Conversation

reedy
Copy link
Member

@reedy reedy commented Aug 16, 2022

@jdforrester
Copy link
Member

Leaving to the Language Engineering team to decide.

@cscott
Copy link
Contributor

cscott commented Aug 18, 2022

I'm going to be bold here, too: since this is merged into core, it should live here too. Language team can revert (here and in core) if there's an issue, but IMO fallbacks here are just a matter of standards-compliance.

@cscott cscott merged commit 70b5ee2 into master Aug 18, 2022
@reedy reedy deleted the reedy-patch-1 branch August 18, 2022 14:35
wmfgerrit pushed a commit to wikimedia/mediawiki that referenced this pull request Aug 31, 2022
Fix the following failures:

* intersection-observer:
  Partial undo of 803b873, local docs-only change to vendor files.

* jquery.i18n:
  - jquery.i18n.messagestore.js:
    Partial undo 7a4df9b (I83bbbb150), comment change to vendor file.
  - jquery.i18n.fallbacks.js:
    This was changed locally in 6839b85 (I666fbdea8) in July
    and upstreamed by @reedy in
    wikimedia/jquery.i18n#243.
    Since there is only one other patch upstream, a cosmetic one,
    since 1.0.7, I've changed it to an unreleased commit hash upstream
    so that at least the registration roundtrips cleanly. We can update
    to the proper release when we're ready for that (there's something
    to figure out w.r.t plural rule parsing).

* moment: Rename en-sg.js to match package.

* jquery.cookie: Recognise this as the fork that it is. Comment-out
  for now to exclude it from validation since it can't pass.
  We should probably just move this to src/ and embed it within
  mw.cookie or something.

Bug: T203694
Change-Id: Ie705070c695e5e4ab95b51ceb434a0eadfa0f733
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants