summaryrefslogtreecommitdiffstats
path: root/llvm/docs/Phabricator.rst
Commit message (Collapse)AuthorAgeFilesLines
* phab doc: Replace or remove references to svnSylvestre Ledru2020-01-091-7/+6
|
* phab doc: also document 'arc land'Sylvestre Ledru2020-01-091-0/+6
|
* phab doc: remove the svn sectionSylvestre Ledru2020-01-091-14/+0
|
* Add discussion of git-format-patch to Phabricator.htmlKit Barton2019-12-031-0/+3
| | | | | | | | | | | | | | Summary: There is a discussion of git-format-patch in GettingStarted guide, but no mention of it in the Phabricator.html page. Reviewers: jyknight, delcypher Reviewed By: delcypher Subscribers: llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D69323
* Update GettingStarted guide to recommend that people use the newJames Y Knight2019-01-141-33/+34
| | | | | | | | | | | | | | | | | | | | | official Git repository. Remove the directions for using git-svn, and demote the prominence of the svn instructions. Also, fix a few other issues while I'm in there: * Mention LLVM_ENABLE_PROJECTS more. * Getting started doesn't need to mention test-suite, but should mention clang and the other projects. * Remove mentions of "configure", since that's long gone. I've also adjusted a few other mentions of svn to point to github, but have not done so comprehensively. Differential Revision: https://reviews.llvm.org/D56654 llvm-svn: 351130
* Introduce bug life cycle documentation.Kristof Beyls2018-11-071-0/+6
| | | | | | | | | | | | | | | Document what is expected during: * triaging * actively working on a bug * closing/resolving Also document how we maintain: * product/component breakdown * default-cc lists per component Differential Revision: https://reviews.llvm.org/D53691 llvm-svn: 346299
* [docs] Tweak update to Phabricator docs about setting repository for diffs ↵Ben Hamilton2018-01-121-4/+6
| | | | | | | | | | | | | | | | | | uploaded via web Summary: In D41919, I missed that there was a *second* step when uploading diffs via web where the repository should be specified. Reviewers: asb, probinson Reviewed By: asb Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D41956 llvm-svn: 322375
* [docs] Update Phabricator docs about setting repository for diffs uploaded ↵Ben Hamilton2018-01-111-5/+5
| | | | | | | | | | | | | | | | via web Summary: Docs are out of date now that we have separate repositories for LLVM, Clang, etc. Reviewers: asb Reviewed By: asb Differential Revision: https://reviews.llvm.org/D41919 llvm-svn: 322290
* [Docs] Add Contributing page.Florian Hahn2018-01-041-0/+2
| | | | | | | | | | | | | | | | | | | | | | This new page acts as an entry point for (new) contributors to LLVM and provides information about * What to contribute * How to submit a patch * Where to start to learn more about LLVM's architecture and internals. This version of the page duplicates some information from the DeveloperPolicy and Phabricator pages. Subsequent changes should work towards moving information for new developers to this page, where it makes sense. Reviewers: reames, probinson, kristof.beyls, silvas, rengolin, asb Reviewed By: silvas Differential Revision: https://reviews.llvm.org/D41665 llvm-svn: 321804
* docs/Phabricator: Better git examples to produce full context patchesMatthias Braun2017-06-151-1/+2
| | | | llvm-svn: 305515
* [doc] Fix minor grammatical error in Phabricator.rstAnmol P. Paralkar2017-01-051-1/+1
| | | | | | | | | | | | Summary: Test commit, fix minor grammatical error in Phabricator.rst Reviewers: delcypher Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D28214 llvm-svn: 291101
* [doc] Clarify steps for contributors without commit access.Florian Hahn2016-12-301-2/+6
| | | | | | | | | | | | Summary: Update the Phabricator docs to clarify how changes are merged for contributors without commit access. Reviewers: delcypher, aaron.ballman Subscribers: aaron.ballman, anmol, llvm-commits Differential Revision: https://reviews.llvm.org/D28184 llvm-svn: 290767
* [docs] Improve the documentation on committing code reviewed onDan Liew2016-01-141-17/+60
| | | | | | | | | | | | | | | | | | | | | | Phabricator to trunk. The previous documentation had a few issues: * It did not make it explicit that code could be committed without using the Arcanist tool and how this should be done. * There was also an implicit assumption on using Subversion rather than git-svn in the example using Arcanist. The documentation now explicitly mentions both cases and details how to commit to trunk in each case. Reviewers: klimek, probinson Subscribers: probinson, nwilson, reames, llvm-commits Differential Revision: http://reviews.llvm.org/D15801 llvm-svn: 257764
* How to close a review manually.Paul Robinson2016-01-081-1/+7
| | | | llvm-svn: 257180
* Add advice on choosing reviewersPaul Robinson2015-12-221-1/+19
| | | | llvm-svn: 256265
* docs: Fix typo in Phabricator.rstReid Kleckner2015-08-061-1/+1
| | | | | | | | | | | | Summary: Fix a small typo in Phabricator.rst: s/your/you. Reviewers: rnk Subscribers: rnk, llvm-commits Differential Revision: http://reviews.llvm.org/D11585 llvm-svn: 244231
* Rename all references to old mailing lists to new lists.llvm.org address.Tanya Lattner2015-08-051-1/+1
| | | | llvm-svn: 243999
* docs: update arcanist linksMartell Malone2015-07-281-2/+2
| | | | | | | | | | | | | | Summary: I need a test commit for using arc. This seems like an appropriate commit to use as a test We may want to port this commit back to 3.7 also Subscribers: llvm-commits Differential Revision: http://reviews.llvm.org/D11527 llvm-svn: 243408
* Expand Phabricator docs slightlyScott Douglass2015-07-011-7/+9
| | | | llvm-svn: 241161
* Add some more detailed docs about the current state of Phabricator andChandler Carruth2015-05-271-1/+11
| | | | | | reviwes.llvm.org to help reduce confusion. llvm-svn: 238295
* [docs] add cross-referenceScott Douglass2015-03-311-1/+3
| | | | llvm-svn: 233699
* Explain how to abandon a review.Paul Robinson2015-03-301-1/+14
| | | | llvm-svn: 233600
* Phabricator calls it "subscriber" not "cc"Paul Robinson2015-01-131-1/+2
| | | | llvm-svn: 225747
* Phabricator doc: Explicit the fact that the patch needs to be there before ↵Sylvestre Ledru2014-07-041-0/+1
| | | | | | the commit llvm-svn: 212328
* Also document the 'arc commit' commands in the 'Committing a change' section ↵Sylvestre Ledru2014-07-021-0/+8
| | | | | | of the Phabricator doc llvm-svn: 212184
* Changed Phab 'CC' to 'subscriber'; fixed typoSanjay Patel2014-06-261-3/+3
| | | | llvm-svn: 211793
* Mention that Phabricator users should subscribe to *-commitsReid Kleckner2014-06-251-4/+16
| | | | | | | This probably explains why a lot of messages get lost for first time Phabricator users. llvm-svn: 211731
* Update of the documentation: I think we are now happy with PhabricatorSylvestre Ledru2014-05-141-2/+1
| | | | llvm-svn: 208764
* Make docs point to new domain.Manuel Klimek2014-04-071-3/+3
| | | | llvm-svn: 205701
* Docs: remove paragraph about manual account creation.Tim Northover2014-03-101-7/+0
| | | | | | | | There's now a normal UI for that, apparently. Patch by Manuel Jacob. llvm-svn: 203481
* Add a note about using "Differential Revision:" in commit messagesMark Seaborn2014-02-111-0/+20
| | | | | | | | | | I noticed this convention from the commit logs. It seems like it would be useful to document it, to encourage other committers to link back to code reviews in their commits. Differential Revision: http://llvm-reviews.chandlerc.com/D2678 llvm-svn: 201160
* Update of the 'Code Review' page. The link is now called 'Create Diff' ↵Sylvestre Ledru2013-11-111-1/+1
| | | | | | instead of 'Create Revision' llvm-svn: 194378
* Phabricator.rst: tiny fixHans Wennborg2013-07-251-1/+1
| | | | llvm-svn: 187164
* Update phab docs to clarify how to accept a change.Manuel Klimek2013-02-131-0/+6
| | | | llvm-svn: 175047
* docs: use code font for console commands in phabricator manualAlexey Samsonov2012-11-061-2/+2
| | | | llvm-svn: 167459
* Clarify that there is an option other than OAuth for Phabricator andChandler Carruth2012-10-271-4/+10
| | | | | | | | offer up my email to the spam lords for it. Hopefully this will eventually be more automatic, but we don't want people to think there is only one option. llvm-svn: 166870
* docs: Minor clean up of Phabricator documentation.Sean Silva2012-10-121-7/+5
| | | | llvm-svn: 165779
* Adds Phabricator documentation. This is a first step that answers many ↵Manuel Klimek2012-10-111-0/+96
questions we have seen. llvm-svn: 165736
OpenPOWER on IntegriCloud