mirror of
https://github.com/Jermolene/TiddlyWiki5
synced 2024-07-07 04:24:22 +00:00
55 lines
2.9 KiB
Plaintext
55 lines
2.9 KiB
Plaintext
created: 20131101111400000
|
|
modified: 20210402095728684
|
|
tags: Community
|
|
title: Contributing
|
|
type: text/vnd.tiddlywiki
|
|
|
|
Here we focus on contributions via GitHub Pull Requests but there are many other ways that anyone can help the TiddlyWiki project, such as [[reporting bugs|ReportingBugs]] or helping to [[improve our documentation|Improving TiddlyWiki Documentation]].
|
|
|
|
! Rules for Pull Requests
|
|
|
|
PRs must meet these minimum requirements before they can be considered for merging:
|
|
|
|
* The material in the PR must be free of licensing restrictions. Which means that either:
|
|
** The author must hold the copyright in all of the material themselves
|
|
** The material must be licensed under a license compatible with TiddlyWiki's BSD license
|
|
* The author must sign the Contributors License Agreement (see below)
|
|
* Each PR should only make a single feature change
|
|
* The title of the PR should be 50 characters or less
|
|
* The title of the PR should be capitalised, and should not end with a period
|
|
* The title of the PR should be written in the imperative mood. See below
|
|
* Adequate explanation in the body of the PR for the motivation and implementation of the change. Focus on the //why// and //what//, rather than the //how//
|
|
* PRs must be self-contained. Although they can link to material elsewhere, everything needed to understand the intention of the PR should be included
|
|
* Documentation as appropriate for end-users or developers
|
|
* Observe the coding style
|
|
* Read the developers documentation
|
|
* Please open a consultation issue prior to investing time in making a large PR
|
|
|
|
The "imperative mood" means written as if giving a command or instruction. See [[this excellent post|https://chris.beams.io/posts/git-commit/#imperative]] for more details, but the gist is that the title of the PR should make sense when used to complete the sentence "If applied, this commit will...". So for example, these are good PR titles:
|
|
|
|
* If applied, this commit will //update the contributing guidelines//
|
|
* If applied, this commit will //change css-escape-polyfill to a $tw.utils method//
|
|
* If applied, this commit will //make it easier to subclass the wikitext parser with a custom rule set//
|
|
|
|
These a poorly worded PR titles:
|
|
|
|
* ~~If applied, this commit will //edit text widgets should use default text for missing fields//~~
|
|
* ~~If applied, this commit will //signing the CLA//~~
|
|
* ~~If applied, this commit will //don't crash if options.event is missing//~~
|
|
|
|
PR titles may also include a short prefix to indicate the subsystem to which they apply. For example:
|
|
|
|
* //Menu plugin: Include menu text in aerial rotator//
|
|
|
|
! Contributor License Agreement
|
|
|
|
{{Contributor License Agreement}}
|
|
|
|
! How to sign the CLA
|
|
|
|
{{Signing the Contributor License Agreement}}
|
|
|
|
---
|
|
|
|
//The CLA documents used for this project were created using [[Harmony Project Templates|http://www.harmonyagreements.org]]. "HA-CLA-I-LIST Version 1.0" for "CLA-individual" and "HA-CLA-E-LIST Version 1.0" for "CLA-entity".//
|