PageAssessments has Invalid composer.json
Closed, ResolvedPublic
Actions

Description

$ composer validate
./composer.json is valid for simple usage with composer but has
strict errors that make it unable to be published as a package:
See https://getcomposer.org/doc/04-schema.md for details on the schema
name : The property name is required
description : The property description is required
No license specified, it is recommended to do so. For closed-source software you may use "proprietary" as license.

I think we should fix these issues so you can at least specify the extension manually.

It should specify a name as mediawiki/page-assessments and a type as mediawiki-extension,

Details

	Subject	Repo	Branch	Lines +/-
	PageAssessments has Invalid composer.json	mediawiki/extensions/PageAssessments	master	+1 -0
	Invalid composer.json in PageAssessments extension	mediawiki/extensions/PageAssessments	master	+3 -0

Customize query in gerrit

Related Objects

Mentioned In: T178137: MediaWiki-Vendor creates a scenario in which incompatible versions of dependencies can be present
T170897: Setup a Composer Repository (Packagist) for MediaWiki Extensions
Mentioned Here: T170897: Setup a Composer Repository (Packagist) for MediaWiki Extensions
T166956: Cannot use Composer's CLI to manage a project's dependencies

Event Timeline

dbarratt created this task.Jun 7 2017, 5:23 PM

Restricted Application added subscribers: TerraCodes, Aklapper. · View Herald TranscriptJun 7 2017, 5:23 PM

Eugene233 added a project: Africa-Wikimedia-Developers.Jul 16 2017, 2:34 AM

Eugene233 moved this task from Backlog to New Contributor's Tasks (Incoming) on the Africa-Wikimedia-Developers board.Jul 16 2017, 2:36 AM

rosalieper claimed this task.Jul 16 2017, 10:33 PM

Change 365541 had a related patch set uploaded (by Rosalieper; owner: Rosalieper):
[mediawiki/extensions/PageAssessments@master] Invalid composer.json in PageAssessments extension

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

gerritbot added a project: Patch-For-Review.Jul 17 2017, 12:57 AM

It should specify a name as mediawiki/page-assessments and a type as mediawiki-extension,

Name should be "PageAssessments"
Type should be "parserhook"

In accordance with how other extensions do this.

In T167329#3442401, @Niharika wrote:

It should specify a name as mediawiki/page-assessments and a type as mediawiki-extension,

Name should be "PageAssessments"
Type should be "parserhook"

In accordance with how other extensions do this.

I'm talking about the name in composer.json I looked at other extensions at that was the format for the type which is always mediawiki/extension-name and the type is always mediawiki-extension per Composer Installers

In T167329#3444434, @dbarratt wrote:

In T167329#3442401, @Niharika wrote:

It should specify a name as mediawiki/page-assessments and a type as mediawiki-extension,

Name should be "PageAssessments"
Type should be "parserhook"

In accordance with how other extensions do this.

I'm talking about the name in composer.json I looked at other extensions at that was the format for the type which is always mediawiki/extension-name and the type is always mediawiki-extension per Composer Installers

My bad. I was looking at extension.json instead. You're right.

The premise here is invalid, it validates just fine if you provide the --no-check-publish option. Adding those fields just make it publishable on packagist, which is not something that's recommended.

In T167329#3444770, @Legoktm wrote:

The premise here is invalid, it validates just fine if you provide the --no-check-publish option. Adding those fields just make it publishable on packagist, which is not something that's recommended.

I mean at a minimum I think the name and type should be correct so the extension can be installed with composer, the other fields are not important, but a licensee and description is helpful.

In T167329#3444786, @dbarratt wrote:

I mean at a minimum I think the name and type should be correct so the extension can be installed with composer, the other fields are not important, but a licensee and description is helpful.

Installing extensions via composer isn't recommended (I did just say that...). How exactly is it helpful? I think that duplicating the license and description (hardcoded English in fact) isn't helpful. It's clearly confusing given T167329#3444750.

Change 365541 merged by jenkins-bot:
[mediawiki/extensions/PageAssessments@master] Invalid composer.json in PageAssessments extension

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

In T167329#3444793, @Legoktm wrote:

Installing extensions via composer isn't recommended (I did just say that...). How exactly is it helpful? I think that duplicating the license and description (hardcoded English in fact) isn't helpful. It's clearly confusing given T167329#3444750.

I though you were saying publishing to packagist was not recommendend, not that installing an extension via composer is not recommended (which does not require the extension to be on Packagist).

There is a larger discussion on T166956 I really think Composer support should be available but completely optional .

I'm fine leaving out the description and a license as that does not prevent installation with composer. An invalid name and type does.

I created T170897: Setup a Composer Repository (Packagist) for MediaWiki Extensions as an alternative way to resolve the issue without modifying this extension's repository.

xSavitar moved this task from New Contributor's Tasks (Incoming) to Tasks in progress (Doing) on the Africa-Wikimedia-Developers board.Jul 18 2017, 1:46 AM

Change 372802 had a related patch set uploaded (by Rosalieper; owner: Rosalieper):
[mediawiki/extensions/PageAssessments@master] PageAssessments has Invalid composer.json

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

dbarratt mentioned this in T170897: Setup a Composer Repository (Packagist) for MediaWiki Extensions.Aug 23 2017, 7:24 PM

Change 372802 merged by jenkins-bot:
[mediawiki/extensions/PageAssessments@master] PageAssessments has Invalid composer.json

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

ReleaseTaggerBot added a project: MW-1.30-release-notes (WMF-deploy-2017-08-29 (1.30.0-wmf.16)).Aug 25 2017, 10:00 PM