Skip to main content

Documentation Pull Request Acceptance Workflow

One Time Setup

  • Fork the tutorial repository.
  • Clone your forked repository.
    $ git clone https://github.com/YOUR-USERNAME/jakartaee-firstcup.git
    
  • Configure the remote for your fork.
    $ git remote add upstream https://github.com/eclipse-ee4j/jakartaee-firstcup.git
    $ git remote -v
    origin    https://github.com/YOUR-USERNAME/jakartaee-firstcup.git (fetch)
    origin    https://github.com/YOUR-USERNAME/jakartaee-firstcup.git (push)
    upstream    https://github.com/eclipse-ee4j/jakartaee-firstcup.git (fetch)
    upstream    https://github.com/eclipse-ee4j/jakartaee-firstcup.git (push)
    

    Raising a Pull Request

  • Sync the master of your fork with upstream master.
    $ git fetch upstream
    $ git checkout master
    $ git merge upstream/master
    $ git push origin master # push local master to github fork.
    
  • Create a local topic branch in your fork from your master.
    $ git checkout -b doc_update
    
  • Do the development in your branch.
  • Commit all the changes.
    $ git add src/main/jbake/content/my.adoc
    $ git commit -m "my commit message"
    
  • Push your changes in a remote branch of your fork.
    $ git push origin doc_update
    
  • Before raising a Pull Request, please raise an issue if it doesn’t exist. We would like every Pull Request to be associated with an issue. Submit the Pull Request referring to the issue number.
  • Raise a Pull Request.
  • Make sure you put a proper ‘title’ for the Pull Request. The title of the Pull Request would become the commit message. Instead of giving ‘title’ like “Iss xxxx” or “Fixes #xxxxx”, consider giving a proper one line ‘title’ for the Pull Request like “Fixes xxx : <brief description about the issue/fix>”
  • In the Pull Request description (body), please mention “Fixes #xxxxx” in order to link the Pull Request with the Issue you are fixing.
  • If you have signed the ECA, one of the project team members will review your Pull Request.

Back to the top