2015-03-23

Created page with "== Justificación =="

New page

<languages/>

These are guidelines for extracting, publishing and managing libraries based on code originally developed as a part of MediaWiki or a related Wikimedia project.

Some of these are PHP-specific, but others are general and apply to all languages.

== Justificación ==

There is a growing desire to separate useful libraries from the core MediaWiki application and make them usable in non-MediaWiki based projects.

This "''librarization''" of MediaWiki is thought to have several long term advantages:

* Make life better for new (and experienced) developers by organizing the code into simple components that can be easily understood.

* Reverse inertia toward ever expanding monolithic core by encouraging developers (in core) to develop their work as reusable modules with clearly-defined interfaces

* Start making true unit testing of core viable by having individually-testable units

* Provide an interim step on the way to service-oriented architecture in a way that is useful independently of that goal

* Encourage reuse and integration with larger software ecosystem. Done correctly, this will provide a useful means of expanding our development capacity through recruitment of library authors eager to showcase their work on a top 10 website.

* Share our awesome libraries with others and encourage contributions from them even if they aren't particularly interested in making our sites better.

In order for this strategy to be successful, these libraries need to develop a life of their own, independent of MediaWiki.

Therefore, it will be important for library authors to have some latitude and independence in making the library successful.

The policies surrounding these should largely be dictated by the primary maintainer of the library, and the choices made may diverge from MediaWiki core.

Note that that primary maintainer may not be the original author of the majority of the code (or even any of it), and will have latitude to make independent decisions about the library.

The amount of latitude a maintainer gets is proportional to the amount of commitment, credibility and hard work with respect to the library they maintain.

== Repository hosting guidelines ==

* Hosted in {{ll|Gerrit}} with GitHub mirror

* Hosted under the [https://github.com/wikimedia Wikimedia GitHub organization]

* Hosted under another GitHub organization

* Hosted in {{ll|Phabricator}} with GitHub mirror (?? are we ready to try [http://phabricator.org/applications/arcanist/ arc] out with libraries?)

The Wikimedia Foundation will likely invest in tooling that makes code review transfer from GitHub to internal review tools (Phabricator) in the future.

Eventually this should eliminate the difference between hosting the primary git repository with the Wikimedia Foundation or GitHub.

In the near term, Gerrit hosting is the default hosting option except in cases where an effort is being made to attract a significant portion of contributions from external developers.

Don't host under an individual user's GitHub account.

It complicates code review based on pull requests for the repository owner and makes management of the repository by a shared group difficult.

Note: this doesn't mean it has to be hosted under the Wikimedia account specifically; in fact, it can be more convenient to have the project under a different organizational account specific to the project, like [https://github.com/cssjanus CSSJanus].

== Repository naming guidelines ==

Probably varies somewhat based on hosting location. Follow the local conventions as far as reasonable and possible.

'''Do not''':

* Add "mediawiki-" or "wikimedia-" prefixes to GitHub hosted projects to match the Gerrit mirror naming. [[User:Chad (WMF)|Chad]] will taunt you mercilessly if you do.

== Issue tracking guidelines ==

* Phabricator

* GitHub

Your library's issue tracking should match your git hosting, in order to reduce the friction of matching commits and pull requests to issues and vice versa. Thus Gerrit-hosted repos should [[Phabricator|Wikimedia's Phabricator instance]] and GitHub-hosted repos should use GitHub's built-in issue tracker.

== Code review guidelines ==

Project code review should use the tool most closely associated with the primary git hosting. Regardless of choice of hosting platform, pre-merge code review and unit testing are strongly encouraged. Blatant self-merge behavior should be seen just as distasteful on GitHub as it generally is in Gerrit.

If primary hosting is via GitHub, changes should be proposed via pull requests rather than direct push to master. In most cases the pull requests should originate from a fork of the repository associated with the user's own GitHub account. [http://gerrithub.io GerritHub] is a Gerrit powered code review service that can be used with GitHub hosted repositories for projects that want to use Gerrit but for some reason do not want to host with Wikimedia.

== Code style guidelines ==

We encourage [[Manual:Coding_conventions|MediaWiki style]] or [http://www.php-fig.org/psr/psr-2/ PHP PSR-2], but the most important things are clarity, consistency, and best likelihood of adoption by the library's developer community.

Automated style checks are ''STRONGLY'' encouraged for new libraries to reduce bike shedding on reviews and make it easier for new contributors to conform to the project's chosen style. The library's README should also point to a style guide.

MediaWiki style checking can be added to a Composer-managed library using the [//packagist.org/packages/mediawiki/mediawiki-codesniffer mediawiki/mediawiki-codesniffer] package:

<syntaxhighlight lang="javascript" style="overflow:auto;">

{

"require-dev": {

"squizlabs/php_codesniffer": "2.1.*",

"mediawiki/mediawiki-codesniffer": "0.1.0"

},

"scripts": {

"phpcs": [

"phpcs --standard=vendor/mediawiki/mediawiki-codesniffer/MediaWiki --ignore=vendor -v ."

]

}

}

</syntaxhighlight>

Be sure to check and see if 0.1.0 is the latest stable code standard on [https://packagist.org/packages/mediawiki/mediawiki-codesniffer packagist.org].

PSR-2 style checking is built into [//packagist.org/packages/squizlabs/php_codesniffer PHP_CodeSniffer]:

<syntaxhighlight lang="javascript" style="overflow:auto;">

{

"require-dev": {

"squizlabs/php_codesniffer": "~2.1.0"

},

"scripts": {

"phpcs": [

"phpcs --encoding=utf-8 --standard=PSR2 --extensions=php src/* tests/phpunit/*"

]

}

}

</syntaxhighlight>

== Automated testing guidelines ==

Both pre and post-merge testing should be used by libraries. The testing should include basic lint, unit tests and coding convention checks.

* Projects hosted with the WMF can use Jenkins.

** We use an entry point of <code>composer test</code>, which runs lint and unit tests. See [[Continuous integration/Test entry points#PHP]] for more details.

* GitHub hosted projects can use Travis.

** See cssjanus's [https://github.com/cssjanus/php-cssjanus/blob/master/.travis.yml .travis.yml] and [https://github.com/cssjanus/php-cssjanus/blob/master/composer.json composer.json] as an example.

== Packagist guidelines ==

PHP libraries should provide a <tt>composer.json</tt> package manifest and be published on the main composer repository [https://packagist.org/ packagist.org]. When adding new packages:

# Follow the [[Manual:composer.json best practices|composer.json best practices]] and add one to your repository

# Add the package to packagist.org:

#* Use the github.com mirror as the git url, this will allow composer to download zipballs which can be cached

#* Ask an [https://github.com/orgs/wikimedia/people owner in the wikimedia github organization] to set up the packagist.org service hook

# Add the "mediawiki" and "wikimedia" accounts as co-maintainers

The following people have access to the different accounts in case a package needs to be updated for any reason:

* '''mediawiki''': [[User:^demon|^demon]], [[User:hashar|hashar]], [[User:Legoktm|Legoktm]], [[User:BDavis (WMF)|BDavis]], ...? (credentials kept on palladium and accessible by roots)

* '''wikimedia''': [[User:BDavis (WMF)|BDavis]], ...? (credentials kept on palladium and accessible by roots)

== License guidelines ==

For almost anything that gets extracted from MediaWiki, it's likely that it will need to be GPLv2 (or later). '''All''' contributors must agree to a change of license from GPLv2+ in order for anyone to change the license (other than changing to GPLv3). The license of the new library needs to remain clearly marked in the headers of the code, and the full license file (typically called "LICENSE" or "COPYING") must be carried into the new project.

For a library consisting entirely of new code any license complying with the [http://opensource.org/osd Open Source Definition] is likely to be acceptable, but the [http://opensource.org/licenses/gpl-license GPLv2+] and [http://www.apache.org/licenses/LICENSE-2.0 Apache License 2.0] licenses may be the easiest to adopt. Both include contributor copyright grant clauses which are important for ensuring the integrity of the project's code base. The Apache2 license is seen as more ''permissive'' in that it allows derivative works to include a separate license for new contributions.

== Documentation guidelines ==

Any library should have a README file that describes the project at a high

level. Typically this file should be formatted using [http://daringfireball.net/projects/markdown/ Markdown] syntax for headers and links as that has become a common standard for the majority of git repository browsers and is still very human readable.

A good README will include:

* A brief description of the primary use case the library solves

* How to install the library

* How to use the library (prose and brief code example if possible)

* How to contribute

** License name (GPL-2, ...)

** Where to submit bugs

** Where to submit patches

** Link to coding standard

** How to run tests

** Where to see automated test results

== Common combinations ==

* Hosted in Gerrit and published to Packagist under the <code>wikimedia</code> namespace

* Hosted in the Wikimedia GitHub organization and published to Packagist under the <code>wikimedia</code> namespace

* Hosted in another GitHub organization and published to Packagist as something other than wikimedia/mediawiki

When hosting is in Gerrit the project should run as any other "typical" Wikimedia sponsored project with Gerrit code review, phabricator bug tracking and wiki documentation.

If hosted at GitHub the project could reasonably choose to do code review via pull requests and host the bug tracker on GitHub. This choice should be considered carefully on a project by project basis as divergenceof code review and issue tracking tools from the larger Wikimedia community has some disadvantages:

* The [[Bugwrangler]] will not be expected to monitor your project for issues.

* Members of the MediaWiki developer community will probably file bugs against your product in Phabricator anyway.

* Moving a bug between your project and MediaWiki will be a more involved process.

These downsides may diminish over time if better bots can be created to integrate between the two environments.

Hosting under an independent GitHub organization makes sense for certain projects (CSSJanus, Wikidata, Semantic MediaWiki, ...) where an effort is being made to develop an independent and sustaining community for the project. It is especially reasonable in the case of a library like CSSJanus that is attempting to establish a cross-community and cross-language standard set of tools where only a portion of the tools overlap with the Wikimedia universe.

== Transferring an existing GitHub repo to Wikimedia ==

* File a ticket in the <span class="plainlinks">[//phabricator.wikimedia.org/maniphest/task/create?projects=Librarization&title=GitHub+transfer:&priority=50 Phabricator Librarization]</span> project requesting transfer.

* When contacted, add the responding Wikimedia GitHub administrator to the project as a "Collaborator".

* The administrator will move the project to Wikimedia and give you access.

== Tips for extracting a library ==

The details of extracting a library will vary depending on the code being extracted and its current entanglement with other MediaWiki specific classes. In the best case, the code you want to extract is already contained in the <code>includes/libs</code> directory of mediawiki/core.git and thus completely unencumbered. It is suggested that code which is not in this state is progressively updated and refactored until that is the case.

Once you get all the code into <code>includes/libs</code>, things become a little more straight forward:

# Create a new project following the rest of the guidelines in this RFC.

# Import the code from <code>includes/libs</code> into your new project<ref>[[gerrit:172312]]</ref>.

#* It may be possible to use <code>git filter-branch</code> to extract a copy of the files with commit history preserved, but that is not currently considered a prerequisite for extraction. It should be sufficient to take the current head of the files into the new project and provide documentation of the file provenance in the new project's README<ref>[//github.com/wikimedia/cdb#history github.com/wikimedia/cdb#history]</ref>.

# Create a proper <code>composer.json</code> file that follows [[Manual:composer.json best practices|best practices]] for the project and publish to Packagist<ref>[https://packagist.org/packages/wikimedia/cdb wikimedia/cdb at Packagist]</ref>.

# Tag the repository to create a stable release.

# Propose a change to <code>mediawiki/vendor.git</code> importing the stable release of your new project<ref>[[gerrit:172793]]</ref>. See [[Manual:External libraries]] for additional details.

# Propose a change to <code>composer.json</code> in <code>mediawiki/core.git</code> to require the stable release of your new project<ref>[[gerrit:172779]]</ref>.

It may also be necessary to introduce shim classes <code>mediawiki/core.git</code> to provide a backwards compatible bridge between your extracted library and the existing MediaWiki code base. The [//github.com/wikimedia/cdb CDB] library did this to provide backwards-compatible class names which did not require the use of the new <code>Cdb\</code> namespace<ref>[[gerrit:172779]]</ref>.

== RFC information ==

{{RFC

| created = 2014-12-24

| authors = {{U|BDavis (WMF)}}

| decision = Accepted. [[User:Tim Starling|Tim Starling]] ([[User talk:Tim Starling|talk]]) 21:45, 14 Jan 2015 (UTC)

| status = accepted

}}

This originated as a '''request for comment''', "Guidelines for extracting, publishing and managing libraries". [http://bots.wmflabs.org/~wm-bot/logs/%23wikimedia-office/20150114.txt The decision] was to move this page out of the RFC space and improve it as documentation.

{{clear}}

== See also ==

* [[Manual:Composer.json best practices]]

* [[Requests_for_comment/Composer_managed_libraries_for_use_on_WMF_cluster|Composer managed libraries for use on WMF cluster RFC]]

* [[Library infrastructure for MediaWiki]]

== References ==

<references/>

[[Category:Composer]]

[[Category:MediaWiki Development]]

Show more