inblockio / mediawiki-extensions-Aqua

This MediaWiki extension includes the Aqua implementation. Turning MediaWiki into a powerful versioned notary service with API's for import and export of Data in JSON format.
GNU General Public License v3.0
7 stars 6 forks source link

Upgrade to MW 1.39 #349

Closed it-spiderman closed 1 year ago

it-spiderman commented 1 year ago

Commits should be squashed, so that it appears as a single change

rht commented 1 year ago

@it-spiderman should I wait until you have fixed the CI fails?

rht commented 1 year ago

Is it a deep error, or shallow error that can be iteratively fixed quickly?

it-spiderman commented 1 year ago

Hello @rht Main CI issue is that it not building MW 1.39 and its still using composer 1. TBH, i dont know how to change the CI setup. Can you help here?

rht commented 1 year ago

This code snippet needs to be modernized: https://github.com/inblockio/mediawiki-extensions-Aqua/blob/3cef7b5f57296de683f074ae1b968e31e7ec6d79/.github/workflows/ci.yml#L35-L40.

it-spiderman commented 1 year ago

thank you @rht , i did it in https://github.com/inblockio/mediawiki-extensions-Aqua/commit/8afe38dfe37dc1d79e476ce5a211385a3bd935a8 However, i dont know how to upgrade composer to V2

rht commented 1 year ago

If you don't pin the composer version (e.g. https://github.com/ProfessionalWiki/WikibaseExport/blob/773c22ce26bf756274fbcaf3acc2d1090d84030c/.github/workflows/ci.yml#L33-L39), it might work.

According to https://github.com/shivammathur/setup-php, you could even force it to be v2:

- name: Setup PHP with composer v2
  uses: shivammathur/setup-php@v2
  with:
    php-version: '8.2'
    tools: composer:v2
it-spiderman commented 1 year ago

@rht Thank you, after some more tinkering, all tests pass now

it-spiderman commented 1 year ago

Again, this should be squash merged, commit history is not clean

rht commented 1 year ago

@FantasticoFox squash and merge is not enabled for this repo.

it-spiderman commented 1 year ago

I tried, can be enabled (if you want)

rht commented 1 year ago

Oh, didn't know I had the privilege to enable it.

it-spiderman commented 1 year ago

Whohooo Other steps (tagging and upgrading other repos) should be done soon, so it doesnt lead to mismatches in versions. @FantasticoFox knows the details (by mail)