Page MenuHomePhabricator
Create Task
Maniphest T409489

Move translation updates for Monaco from Github to Gerrit
Closed, ResolvedPublic1 Estimated Story Points
Actions

Description

The Github repository has been moved to Gerrit: https://gerrit.wikimedia.org/g/mediawiki/skins/Monaco
However, translation updates are still pushed to the old repository, which is no longer maintained. They should be moved to the new repository on gerrit instead.

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
Update the location of Monaco Sidebar from GitHub to Gerrittranslatewikimaster+3 -0
Monaco: update git location from Github to Gerrittranslatewikimaster+5 -33
Make message keys consistent and add missing documentationmediawiki/skins/Monacomaster+43 -26
Customize query in gerrit

Related Objects

Event Timeline

SomeRandomDeveloper created this task.Thu, Nov 6, 8:51 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptThu, Nov 6, 8:51 PM
SomeRandomDeveloper mentioned this in T409487: Create project tag for Monaco.Thu, Nov 6, 8:52 PM
Nikerabbit added a subscriber: Raymond.Fri, Nov 7, 8:46 AM
Nikerabbit triaged this task as Low priority.Mon, Nov 10, 9:00 AM
Nikerabbit added projects: LPL Essential (FY2025-26 Q2), Unplanned-Sprint-Work, LPL Projects (Other).
Nikerabbit set the point value for this task to 1.
Nikerabbit moved this task from Backlog to Configuration changes on the translatewiki.net board.
Nikerabbit moved this task from Backlog to Ready for dev on the LPL Essential (FY2025-26 Q2) board.
Raymond added a comment.Mon, Nov 10, 9:30 AM

Fast i18n review:

  • The message documentation is incomplete
  • The messages have inconsistent message keys: "monaco-", "dynamic-", etc.

I would like to see both points fixed.

Nikerabbit subscribed.Mon, Nov 10, 2:49 PM

Will the GitHub repository be marked read-only? In any case, we can start by removing the github config for now.

SomeRandomDeveloper added a comment.Mon, Nov 10, 3:59 PM
In T409489#11358953, @Nikerabbit wrote:

Will the GitHub repository be marked read-only? In any case, we can start by removing the github config for now.

At some point, it probably will be archived, but the maintainer hasn't been active recently.

In T409489#11357517, @Raymond wrote:

Fast i18n review:

  • The message documentation is incomplete
  • The messages have inconsistent message keys: "monaco-", "dynamic-", etc.

I would like to see both points fixed.

I'll look into creating a patch for that.

SomeRandomDeveloper added a comment.Mon, Nov 10, 4:02 PM

dynamic-links appears to be used to create links (similarly to MediaWiki:Sidebar), so renaming that message would break the feature on all wikis using it.

gerritbot added a comment.Mon, Nov 10, 4:10 PM

Change #1203482 had a related patch set uploaded (by SomeRandomDeveloper; author: SomeRandomDeveloper):

[mediawiki/skins/Monaco@master] Make message keys consistent and add missing documentation

https://gerrit.wikimedia.org/r/1203482

gerritbot added a project: Patch-For-Review.Mon, Nov 10, 4:10 PM
gerritbot added a comment.Wed, Nov 26, 7:24 AM

Change #1211547 had a related patch set uploaded (by Wangombe; author: Wangombe):

[translatewiki@master] Monaco: update git location from Github to Gerrit

https://gerrit.wikimedia.org/r/1211547

Wangombe changed the task status from Open to In Progress.Wed, Nov 26, 7:24 AM
Wangombe claimed this task.
Wangombe moved this task from Ready for dev to In Progress on the LPL Essential (FY2025-26 Q2) board.
gerritbot added a comment.Wed, Nov 26, 11:11 AM

Change #1211547 merged by jenkins-bot:

[translatewiki@master] Monaco: update git location from Github to Gerrit

https://gerrit.wikimedia.org/r/1211547

Nikerabbit added a comment.Wed, Nov 26, 11:30 AM

@Raymond will you handle possible message renames and should we merge https://gerrit.wikimedia.org/r/c/1203482 now?

Raymond added a comment.Wed, Nov 26, 11:32 AM
In T409489#11408971, @Nikerabbit hat geschrieben:

@Raymond will you handle possible message renames and should we merge https://gerrit.wikimedia.org/r/c/1203482 now?

I was a bit too fast in renaming existing translations (done 10 minutes ago) because I thought https://gerrit.wikimedia.org/r/c/1203482 was merged already. i gave +2 one minute ago.

gerritbot added a comment.Wed, Nov 26, 11:40 AM

Change #1203482 merged by jenkins-bot:

[mediawiki/skins/Monaco@master] Make message keys consistent and add missing documentation

https://gerrit.wikimedia.org/r/1203482

Raymond closed this task as Resolved.Wed, Nov 26, 12:05 PM

Renames done, config deployed.

Nikerabbit moved this task from In Progress to Done on the LPL Essential (FY2025-26 Q2) board.Wed, Nov 26, 12:11 PM

Thank you!

Nikerabbit removed a project: Patch-For-Review.Wed, Nov 26, 12:11 PM
A_smart_kitten added a project: Monaco.Wed, Nov 26, 6:12 PM
gerritbot added a comment.Thu, Dec 4, 6:18 AM

Change #1214971 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[translatewiki@master] Update the location of Monaco Sidebar from GitHub to Gerrit

https://gerrit.wikimedia.org/r/1214971

gerritbot added a project: Patch-For-Review.Thu, Dec 4, 6:18 AM
gerritbot added a comment.Thu, Dec 4, 6:28 AM

Change #1214971 merged by jenkins-bot:

[translatewiki@master] Update the location of Monaco Sidebar from GitHub to Gerrit

https://gerrit.wikimedia.org/r/1214971

Maintenance_bot removed a project: Patch-For-Review.Thu, Dec 4, 6:30 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits