Page MenuHomePhabricator

Add additional git, npm and composer workflow documentation to WikibaseLexeme.git readme
Closed, ResolvedPublic5 Estimated Story Points

Description

Currently there are a few workflows used in the development of WikibaseLaxeme that remain undocumented, these include: linting & codesniffing, producing static assests for local development, and running unit / integration tests. In order to allow for clear and transparent workflow process and ease onboarding to the codebase, running instructions for these should be included in the README or in a dedicated Development instructions markdown file linked from the main readme.

Acceptance Criteria:

Developer documentation includes:

  • Updating local npm and composer dependencies
  • Updating submodules
  • Producing static assests for local development
  • Linting / codesniffing instructions
  • Improve Unit / integration test running instructions

Event Timeline

Prio Notes:

  • Does not affect end users
  • Allows current developers to be reminded of common workflows in the repository → Affects development efforts
  • Does not affect known process owners
  • Assists with the onboarding of future developers into this project as maintainers → Affects onboarding efforts, external developers
karapayneWMDE updated the task description. (Show Details)
karapayneWMDE set the point value for this task to 5.

Task Breakdown Notes:

  • The updating submodules and dependencies ACs are really about day-to-day dev workflows, and can be included in their own sections
  • While some unit and integration test instructions are included, they could be reworked a bit to resemble the instructions on new-lexeme-special-page.git
  • As the text in the main README file is getting quite long, we might want to consider splitting the developer guide into a new file and linking to it from the main README
  • We could also potentially add a TOC generator like DocToc to ensure easy navigation of the page

See the following developer guides for reference:

Change 802768 had a related patch set uploaded (by Noa wmde; author: Noa wmde):

[mediawiki/extensions/WikibaseLexeme@master] Rework dev setup and test documentation in README

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

Change 802768 merged by jenkins-bot:

[mediawiki/extensions/WikibaseLexeme@master] Rework dev setup and test documentation in README

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