Skip to content

Latest commit

 

History

History
88 lines (56 loc) · 3.17 KB

CONTRIBUTING.md

File metadata and controls

88 lines (56 loc) · 3.17 KB
Raw

Want to contribute?

That's awesome, thank you!

The following is a set of guidelines for contributing to the MEIGarage. These are mostly guidelines, not rules. Use your best judgment, and feel free to propose changes to this document in a pull request.

I don't want to read this whole thing I just have a question!!!

The best way to initially get in touch with the MEIGarage community is via the mei-garage Slack channel on music-encoding.slack.com. Or you can start a discussion here at GitHub Discussions.

Bug reports and feature requests

If you've noticed a bug or have a feature request, please open a GitHub issue (well, you might want to look through the open issues before to avoid creating duplicates)! It's generally best if you get confirmation of your bug or approval for your feature request this way before starting to code.

Create a branch

If this is something you think you can fix, then create a branch with a descriptive name, either in a [fork of this repo] (if it is clear that you will be the only one working on issue in the new branch), or in this repo directly.

Please keep in mind that before coding along there should be a ticket for the task you want to accomplish.

A good branch name would be (where issue #33 is the ticket you're working on):

git checkout -b issue-33

Build and test your branch locally

With Docker installed you simply tell it to build your locally checked out branch ("issue-33") with

docker build --tag meigarage:issue-33 .

and then you run it like

docker run --name meigarage-issue-33 -p 8080:8080 -d meigarage:issue-33

Make a Pull Request

If everything works as expected, you should switch back to your dev branch and make sure it's up to date with our dev (= our default) branch:

git remote add upstream [email protected]:edirom/MEIGarage.git
git checkout dev
git pull upstream dev

Then update your feature branch from your local copy of develop, and push it

git checkout issue-33
git rebase dev
git push --set-upstream origin issue-33

Finally, go to GitHub and make a Pull Request

Merging a PR (maintainers only)

A PR can only be merged into main by a maintainer if:

  • The branch is building ok on DockerHub.
  • It has been approved by at least two people. If it was a maintainer who opened the PR, only one extra approval is needed.
  • It has no requested changes.
  • It has no conflicts with current develop. (Even better: it is up to date with current develop)

Any maintainer is allowed to merge a PR if all of these conditions are met.

Acknowledgements

Large portions of these Contributing Guidelines were copied from ActiveAdmin. Thanks!