summaryrefslogtreecommitdiff
path: root/CONTRIBUTING.md
Commit message (Collapse)AuthorAgeFilesLines
* Remove FE team labelSean McGivern2017-05-051-3/+6
|
* Improving copy of CONTRIBUTING.md, PROCESS.md, and code_review.mddocs/30141-improve-contributing-guideRémy Coutable2017-05-041-13/+10
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Update ToC of CONTRIBUTING.md and PROCESS.mdRémy Coutable2017-05-031-3/+2
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Improve the Code review guidelines documentationRémy Coutable2017-05-031-18/+0
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Improve the merge request guidelines and DoD in CONTRIBUTING.mdRémy Coutable2017-05-031-11/+15
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Move retro/kickoff doc to PROCESS.mdRémy Coutable2017-05-031-13/+0
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Document labels in CONTRIBUTING.mdRémy Coutable2017-05-021-34/+146
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'improve-mr-reviews-in-contributing' into 'master' Jacob Schatz2017-03-281-7/+9
|\ | | | | | | | | Improve the MR guidelines from the CONTRIBUTING guide See merge request !9967
| * Improve the MR guidelines from the CONTRIBUTING guideimprove-mr-reviews-in-contributingRémy Coutable2017-03-161-7/+9
| | | | | | | | | | | | [ci skip] Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Creates Frontend Style guideFilipa Lacerda2017-03-221-4/+4
| |
* | Add policy for closing stale merge requestsstale-merge-requestsAdam Niedzielski2017-03-161-0/+7
|/
* Describe polling with ETag cachingsh-expand-on-definition-of-doneAdam Niedzielski2017-03-141-2/+3
|
* Add performance/scalability concerns to CONTRIBUTING.mdStan Hu2017-03-131-0/+7
| | | | [ci skip]
* Add a section on issue/MR labeling in CONTRIBUTING.mddocs/contributing-md-labelsAchilleas Pipinellis2017-03-101-0/+5
| | | | | | Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/25097 [ci skip]
* Document "How to get your MR reviewed, approved, and merged"Rémy Coutable2017-03-091-4/+30
| | | | | | [ci skip] Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'cla-first' into 'master' Achilleas Pipinellis2017-03-011-28/+37
|\ | | | | | | | | But notice first. See merge request !9364
| * Add CLA on top of CONTRIBUTING.mdAchilleas Pipinellis2017-03-011-28/+35
| | | | | | | | [ci skip]
| * But notice first.cla-firstsytses2017-02-191-8/+10
| |
* | Switched CONTRIBUTING.md style guide recommendation for method chainingJames Edwards-Jones2017-02-231-1/+1
|/
* Document better the CE -> EE mergedocument-ce-to-eeRémy Coutable2017-02-161-8/+10
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fix menu link in CONTRIBUTINGadam-contributing-fix-linkAdam Niedzielski2017-02-091-1/+1
|
* Merge branch 'patch-12' into 'master' Rémy Coutable2017-02-081-1/+1
|\ | | | | | | | | Fix link in CONTRIBUTING.md See merge request !8859
| * Fix link in CONTRIBUTING.mdCarlos Galarza2017-01-281-1/+1
| |
* | add the stewardship label to contributing.mdstewardship-labelJob van der Voort2017-02-061-0/+16
| |
* | Document that the retro and kickoff notes are publicretro-and-kickoff-are-publicRémy Coutable2017-01-311-0/+21
|/ | | | | | [ci skip] Signed-off-by: Rémy Coutable <remy@rymai.me>
* Remove "official merge window" from CONTRIBUTING.md [ci skip] Adam Niedzielski2017-01-261-8/+0
| | | | | We should not discourage contributors from submitting merge requests anytime they want. Our process for reviewing community contributions no longer depends on the release schedule.
* Update CONTRIBUTING.md after merging "up-for-grabs" and "Accepting Merge ↵contributing-md-accepting-merge-requestsAdam Niedzielski2017-01-261-14/+14
| | | | Requests" [ci skip]
* Link to k8s contributing doc.Sytse Sijbrandij2016-12-271-2/+3
|
* Update CONTRIBUTING.md - Adding info about issues within a milestoneTaurie Davis2016-11-161-1/+4
|
* [ci skip] Fix UX Guide link on Contributing.mdux-guide-link-update-contributingawhildy2016-11-151-4/+2
|
* Merge branch 'rs-bin-changelog' into 'master' Rémy Coutable2016-11-021-7/+2
|\ | | | | | | | | Add a bin/changelog script and changelog documentation See merge request !7098
| * Add changelog documentationRobert Speicher2016-10-311-7/+2
| |
* | Replace MR Description Format links Daniel Axelrod2016-11-021-3/+5
|/ | | | | | | | Replace links to the `Merge request description format` section with up-to-date instructions on using the MR template built into the UI. The `Merge request description format` section was removed in 2f7e28d1, but links were left in the table of contents and in the MR description instructions.
* Convert CHANGELOG to MarkdownRobert Speicher2016-10-161-1/+1
| | | | | | | | | All this does is convert the version sections into headers. The list items shouldn't really be indented by two spaces, but it makes no difference to the rendering and this way we retain authorship history for the actual changes. Related to https://gitlab.com/gitlab-org/release-tools/merge_requests/29
* Improve the contribution and MR review guideRémy Coutable2016-10-071-28/+21
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'sh-add-link-to-reliable-capybara-specs' into 'master' Achilleas Pipinellis2016-09-161-0/+2
|\ | | | | | | | | | | | | Add link to article on how to write reliable, asynchronous tests with Capybara This article has been incredibly useful for all developers. Adding a link to it so more people can refer to it. See merge request !6312
| * Add link to article on how to write reliable, asynchronous tests with Capybarash-add-link-to-reliable-capybara-specsStan Hu2016-09-121-0/+2
| | | | | | | | [ci skip]
* | Simplify the 'Implement design & UI elements' section in CONTRIBUTINGupdate-implement-design-ui-elements-in-CONTRIBUTINGRémy Coutable2016-09-071-16/+2
|/ | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'mr-performance-guides' into 'master' Achilleas Pipinellis2016-08-311-0/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added performance guidelines for new MRs ## What does this MR do? This MR adds a set of guides that should be followed by merge request authors. ## Are there points in the code the reviewer needs to double check? Spelling, grammar, etc ## Why was this MR needed? There is no set of guidelines one should follow when submitting merge requests. This leads to developers at times disregarding performance. This in turn results in performance specialists having to clean up the mess, or production engineers being woken up in the middle of the night because the database is on fire. ## Does this MR meet the acceptance criteria? - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - Tests - [x] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) cc @DouweM @rspeicher @pcarranza @dzaporozhets See merge request !5905
| * Added performance guidelines for new MRsmr-performance-guidesYorick Peterse2016-08-221-0/+2
| | | | | | | | | | | | | | These guidelines cover the performance requirement for newly submitted merge requests. These guidelines are put in to place to prevent merge requests from negatively impacting GitLab performance as much as possible.
* | Link to the issue templates in Contributing guideMark Fletcher2016-08-251-2/+2
| | | | | | | | | | | | | | | | Change to use a better title style Add backticks to code segments Spelling and Grammar
* | Reflect template addition in the contributing guideMark Fletcher2016-08-241-56/+3
| | | | | | | | | | | | | | | | | | | | | | | | Adjust bug template to adhere to standard bug report format * https://developer.mozilla.org/en-US/docs/Mozilla/QA/Bug_writing_guidelines * Guidelines use the following format - Summary - Steps to Reproduce - Expected Behaviour - Actual Behaviour
* | Update contribution acceptance criteria with tests requirementsdz-test-requiredDmitriy Zaporozhets2016-08-191-3/+2
| |
* | Add test requirement for new filesDmitriy Zaporozhets2016-08-151-1/+3
|/
* Merge branch 'dz-newlines-styleguide' into 'master' Robert Speicher2016-08-081-0/+2
|\ | | | | | | | | Add newlines styleguide for Ruby code See merge request !5636
| * Add newlines styleguide for Ruby codedz-newlines-styleguideDmitriy Zaporozhets2016-08-051-0/+2
| |
* | Add migration-related tips to the "Merge Request Guidelines" docadd-mr-migration-guidelinesTimothy Andrew2016-08-081-0/+4
| | | | | | | | [ci-skip]
* | Merge branch 'coffescript_docs_migration' into 'master' Rémy Coutable2016-08-041-1/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update docs for CoffeeScript -> JavaScript. ## What does this MR do? Removes some CoffeeScript docs and adds more JS ones ## Are there points in the code the reviewer needs to double check? No ## Why was this MR needed? To migrate some docs ## What are the relevant issue numbers? #20098 ## Screenshots (if relevant) None ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] API support added - Tests - [ ] Added for this feature/bug - [ ] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5433
| * | Update docs for CoffeeScript -> JavaScript.Jacob Schatz2016-07-221-1/+2
| |/
* | link the engineering workflow document from process and contributingjob/update-workflow-linksJob van der Voort2016-08-021-0/+2
|/