Skip to content
Snippets Groups Projects

Add CONTRIBUTING.md

Merged Loïck Bonniot requested to merge contributing into master

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
CONTRIBUTING.md 0 → 100644
150 5. Locally merge (or rebase) the upstream development branch into your topic branch:
151
152 ```bash
153 git pull [--rebase] upstream <dev-branch>
154 ```
155
156 6. Push your topic branch up to your fork:
157
158 ```bash
159 git push origin <topic-branch-name>
160 ```
161
162 7. Open a Pull Request with a clear title and description.
163
164 **IMPORTANT**: By submitting a patch, you agree to allow the project owner to
165 license your work under the same license as that used by the project.
  • CONTRIBUTING.md 0 → 100644
    95
    96 You can rebase. But, please, do it locally (when refactoring your own topic branch).
    97 You don't want to have contributors against you after a missed forced push.
    98
    99 ## Pull requests
    100
    101 Good pull requests (patches, improvements, new features) are a fantastic
    102 help. They should remain focused in scope and avoid containing unrelated
    103 commits.
    104
    105 **Please ask first** before embarking on any significant pull request (e.g.
    106 implementing features, refactoring code, porting to a different language),
    107 otherwise you risk spending a lot of time working on something that the
    108 project's developers might not want to merge into the project.
    109
    110 Please adhere to the coding conventions used throughout a project (indentation,
  • Loïck Bonniot Added 1 commit:

    Added 1 commit:

  • We might replace IRC with slack ? Or migrate to IRC and mention a channel and a server ?

    This merge request seems to be a good start. If nobody is against, I'll merge it this afternoon.

    Edited by Dufour Quentin
  • Dufour Quentin mentioned in commit 14afae25

    mentioned in commit 14afae25

  • Dufour Quentin Status changed to merged

    Status changed to merged

  • Please register or sign in to reply
    Loading