Page MenuHomePhabricator

[Task] Move PropertySuggester extension to gerrit
Closed, ResolvedPublic

Description

Related Objects

StatusSubtypeAssignedTask
ResolvedAddshore
ResolvedAddshore
InvalidNone
DeclinedNone
DeclinedNone
ResolvedJanZerebecki
ResolvedJeroenDeDauw
Resolvedaude
ResolvedJanZerebecki
DuplicateNone
DeclinedNone
ResolvedLadsgroup
ResolvedJanZerebecki
DeclinedNone
Resolvedaude
ResolvedJeroenDeDauw
ResolvedWMDE-leszek
InvalidNone
InvalidNone
ResolvedNone
ResolvedVictorbarbu
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedLucas_Werkmeister_WMDE
DeclinedTarrow
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedNone
ResolvedLadsgroup
ResolvedLucas_Werkmeister_WMDE
ResolvedTarrow
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedItamarWMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedItamarWMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedNone
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedTarrow
ResolvedItamarWMDE
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedItamarWMDE
ResolvedItamarWMDE
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedLucas_Werkmeister_WMDE
ResolvedLadsgroup
ResolvedLadsgroup
ResolvedMtDu
ResolvedLydia_Pintscher
DeclinedLydia_Pintscher
InvalidNone
InvalidNone
ResolvedReedy
ResolvedLegoktm
ResolvedLadsgroup
ResolvedNone
ResolvedJanZerebecki
ResolvedNone
ResolvedJanZerebecki
ResolvedKrinkle
InvalidNone
ResolvedJanZerebecki
ResolvedAddshore

Event Timeline

JanZerebecki raised the priority of this task from to Medium.
JanZerebecki updated the task description. (Show Details)
Jonas renamed this task from Move PropertySuggester extension to gerrit to [Task] Move PropertySuggester extension to gerrit.Aug 13 2015, 4:17 PM
Jonas set Security to None.
greg added a subscriber: Ladsgroup.

Change 354965 had a related patch set uploaded (by Ladsgroup; owner: Amir Sarabadani):
[mediawiki/extensions/PropertySuggester@master] Add .gitreview

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

Change 354980 had a related patch set uploaded (by Ladsgroup; owner: Amir Sarabadani):
[mediawiki/extensions/Wikidata@master] Use PropertySuggester in gerrit instead of github

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

Change 354965 merged by Hashar:
[mediawiki/extensions/PropertySuggester@master] Add .gitreview

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

Change 355120 had a related patch set uploaded (by Ladsgroup; owner: Amir Sarabadani):
[wikidata/build-resources@master] Make builder use gerrit instead of github for PropertySuggester

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

Change 354980 abandoned by Ladsgroup:
Use PropertySuggester in gerrit instead of github

Reason:
Done in I326f3cd4fed8ec07f812a631fc63f82f8f940376

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

Most of it is being done but there are two things needing to be done:

  • Update packagist, I don't how to do it or if it's needed
  • Make jenkins run on this properly.

Change 355124 had a related patch set uploaded (by Ladsgroup; owner: Amir Sarabadani):
[integration/config@master] Add PropertySuggester

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

I guess we would also need to somehow also port the releases over to Gerrit, no?

We can also go with dev-master like Wikibase and other gerrit extensions.

Looks like the gerrit repo admin part is done, right? Can I remove releng-related projects now?

I guess we would also need to somehow also port the releases over to Gerrit, no?

Releases are just git tags. Should be simple to clone the repository and then re-push it.

Change 355124 merged by jenkins-bot:
[integration/config@master] Add PropertySuggester

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

Change 355204 had a related patch set uploaded (by Ladsgroup; owner: Amir Sarabadani):
[mediawiki/extensions/PropertySuggester@master] Add package.json

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

Change 355204 merged by jenkins-bot:
[mediawiki/extensions/PropertySuggester@master] Make jenkins pass on master

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

Change 355120 merged by jenkins-bot:
[wikidata/build-resources@master] Make builder use gerrit instead of github for PropertySuggester

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

All patches are merged but some stuff need to be done before calling this done. I add a checklist at the task's description.

One possible thing to add to the checklist: stop L10n-bot doing updates on github, and make sure it only updates repo on Gerrit. Currently it seems updates are done two boths, which results in both repos diverge more and more :)

Change 357771 had a related patch set uploaded (by Ladsgroup; owner: Amir Sarabadani):
[translatewiki@master] Stop updating PropertySuggester in github.

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

Change 357771 merged by jenkins-bot:
[translatewiki@master] Stop updating PropertySuggester in github.

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

Okay, I closed the only one open PR in github: https://github.com/Wikidata-lib/PropertySuggester/pull/179

I don't think making it a redirect would be a good idea because we will lose lots of discussion that might be useful for future references.

Ladsgroup moved this task from Doing to Done on the Wikidata-Former-Sprint-Board board.