Skip to content

Commit

Permalink
Merge pull request #1 from erik-brangs/patch-1
Browse files Browse the repository at this point in the history
Update CONTRIBUTING.md for move to GitHub
  • Loading branch information
nitind authored Sep 19, 2023
2 parents c0a6c81 + 2f4373b commit 7eb15de
Showing 1 changed file with 6 additions and 15 deletions.
21 changes: 6 additions & 15 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,23 +2,14 @@

Thank you for your interest in this project. Don't forget to sign your Eclipse Contributor Agreement before proposing any changes.

In addition to the changes you're proposing, you should be familiar with Git and using SSH keys with Git.

In addition to the changes you're proposing, you should be familiar with Git, using SSH keys with Git, and may wish to read through https://wiki.eclipse.org/Gerrit as we use the Gerrit system for reviewing and merging contributions.

If you're seeing this repository somewhere other than on git.eclipse.org, it's not the authoritative version used for our releases every quarter. Opening pull requests or submitting changes here will be of no use.

1. Clone the repository using one of the URLs at the bottom of http://git.eclipse.org/c/sourceediting/webtools.sourceediting .
1. Set up your Gerrit access, including a SSH public key, on https://git.eclipse.org/r/ .
1. Take note of your Username as listed on https://git.eclipse.org/r/settings/ (when logged in).
1. Add a `git remote` for gerrit using `ssh://{username}@git.eclipse.org:29418/sourceediting/webtools.sourceediting.git`
1. Ready your changes
1. You should import the master unit test project from the `web/tests/org.eclipse.wst.sse.unittests/` location in the repository. Try to at least run the unit tests corresponding to the project for which you're proposing changes.
1. You can also run the Maven build, supplying using at least Tycho 2.3.0 for the values in `-Dtycho-extras.version=${tychoVersion} -Dtycho.version=${tychoVersion}` and profile `bree-libs` with `-Pbree-libs`.
1. Create a local "topic" branch and commit your changes. Please start your commit messages first line with the word "bug" and the bug number, e.g. `Bug 12345` to refer to bug 12345, both to make it more understandable in the eventual git history and to allow Gerrit to automagically link it to the bug report.
1. Push your changes to the Gerrit remote using refspec `HEAD:refs/for/master`
1. You'll be given a URL like https://git.eclipse.org/r/c/sourceediting/webtools.sourceediting/+/163278 unique to your changes. Please go there and click on ADD REVIEWER, choosing the [project lead](https://projects.eclipse.org/projects/webtools.sourceediting/who), so they get notified of your proposal.
1. If you need to make more changes, find the generated Change-Id on your change's page (e.g. `Change-Id: Ibe9f831a614edfe4e637038c4de0c3a849efb329`), and make sure it is copied as the last line in your updated commit message. Then you can push your commit as before and it will have a history in Gerrit on the same page with the same conversations.
If you're seeing this repository somewhere other than [this GitHub repository](https://github.com/eclipse-webtools-sourceediting/sourceediting), it's not the authoritative version used for our releases every quarter. Opening pull requests or submitting changes there will be of no use.

Hints for pull requests:
- You should import the master unit test project from the `web/tests/org.eclipse.wst.sse.unittests/` location in the repository. Try to at least run the unit tests corresponding to the project for which you're proposing changes.
- You can also run the Maven build, supplying using at least Tycho 2.3.0 for the values in `-Dtycho-extras.version=${tychoVersion} -Dtycho.version=${tychoVersion}` and profile `bree-libs` with `-Pbree-libs`.
- Please refer the GitHub issue in your commit message (e.g #1). If you're fixing an old bug [from Bugzilla](https://bugs.eclipse.org/bugs/), start your commit messages first line with the word "bug" and the bug number, e.g. Bug 12345 to refer to bug 12345 to make it more understandable in the eventual git history.

## Eclipse Development Process

Expand Down

0 comments on commit 7eb15de

Please sign in to comment.