Changes between Version 39 and Version 40 of HowToGit


Ignore:
Timestamp:
May 24, 2019, 2:31:35 AM (5 years ago)
Author:
martinl
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • HowToGit

    v39 v40  
    1616 * fork the GRASS GIS repository, and create feature branch(es) with the changes, and suggest your changes as pull requests.
    1717
    18 First [https://github.com/OSGeo/grass fork the GRASS GIS repo] in the !GitHub UI to `your_GH_account`.
     18=== Workflow for core grass repository ===
    1919
    20 === Workflow for core developers ===
     20''- to be discussed -''
    2121
     22First [https://github.com/OSGeo/grass fork the GRASS GIS repo] in the !GitHub UI to `your_GH_account`. This is the same as what !GitHub documentation suggests. See: [https://help.github.com/en/articles/fork-a-repo Fork a repo] and [https://help.github.com/en/articles/syncing-a-fork Syncing a fork] in !GitHub help.
    2223
    23 ''- to be discussed which way -''
    24 
    25 Add: https://help.github.com/en/articles/connecting-to-github-with-ssh
     24Note: add SSH key, see [https://help.github.com/en/articles/connecting-to-github-with-ssh GitHub documentation].
    2625
    2726One time only:
     
    4140upstream  git@github.com:OSGeo/grass.git (push)
    4241}}}
    43 
    4442
    4543Working with git:
     
    7573NOTE: for different pull requests, simply create different feature branches.
    7674
    77 === Workflow for external contributors ===
    78 
    79 - to be discussed: is this needed at all? Maybe above part is enough -
    80 
    81 First fork the GRASS GIS repo in the !GitHub web interface.
    82 
    83 You clone the GRASS GIS repo and add the fork as an additional remote (or the other way around), this makes merging changes from the GRASS GIS repo easier.
    84 
    85 {{{
    86 # create fork via GitHub Web interface
    87 
    88 # checkout your the osgeo repo
    89 git clone https://github.com/OSGeo/grass.git
    90 
    91 # go to repo dir
    92 cd grass
    93 
    94 # add your fork as another remote
    95 git remote add fork https://github.com/<your_GH_account>/grass.git
    96 
    97 # change the push URL for your fork to SSH if you have it set up
    98 git remote set-url --push fork ssh://git@github.com/<your_GH_account>/grass.git
    99 
    100 # all steps see above, core dev section (branch, edit, commit)
    101 ...
    102 
    103 # push feature branch to your own fork repo of GRASS GIS
    104 
    105 git push fork <feature-branch>
    106 
    107 # create pull request in GitHub Web interface (the link is shown conveniently in the terminal)
    108 }}}
    109 
    110 This way origin is the authoritative source for the code, and additional remotes are forks.
    111 
    112 What is missing above is how you actually update the repo with changes in OSGeo repo master branch.
    113 
    114 Alternative is to clone your fork and add osgeo as another remote ("option 2" - "clone fork" as opposed to "clone osgeo"). This would be the same as what !GitHub documentation suggests. See: [https://help.github.com/en/articles/fork-a-repo Fork a repo] and [https://help.github.com/en/articles/syncing-a-fork Syncing a fork] in !GitHub help.
    115 == Keep your local source code up to date ==
     75==== Keep your local source code up to date ====
    11676
    11777[from https://github.com/OSGeo/gdal/blob/master/CONTRIBUTING.md#working-with-a-feature-branch]
     
    13393
    13494Continue do your changes and commit/push them (ideally to a feature branch, see above).
    135 == Review of pull requests ==
     95
     96==== Review of pull requests ====
    13697
    13798In the review phase, PRs can be commented and modified.
     
    139100TODO add more
    140101
    141 == Merging of pull requests ==
     102==== Merging of pull requests ====
    142103
    143104The CI should run successfully for every commit (chunk of commits in PR to be exact) before it goes into the respective branch.
     
    154115
    155116... based on PR nature.
     117
     118=== Workflow for grass-addons repository ===
     119
     120- to be discussed  -
     121
    156122== Switching between branches ==
    157123
     
    165131
    166132See also https://github.com/OSGeo/gdal/blob/master/CONTRIBUTING.md#backporting-bugfixes-from-master-to-a-stable-branch
     133
    167134== Further reading ==
    168135