Excerpt |
---|
Provides information describing how a committer reviews and approves pull requests. |
...
Table of Contents | ||||
---|---|---|---|---|
|
...
Code Block | ||
---|---|---|
| ||
git config --global user.email myusername@apache.org |
Alternatively, you can edit your .gitconfig file so that you don't need to enter the commands interactively by adding
[user]
name = myusername
email = myusername@apache.org
Step 2: Check trafodion-contributors Group
...
https://github.com/orgs/trafodion-contributors/people
If you are not listed in the above group, send a message to one of the members listed as an owner and request to be added to the group.
Step 3: Set Up Work Space
...
In VNC/Gnome environment, either add to
.bashrc
or type at your current shell:Code Block language bash unset SSH_ASKPASS
- Pushing code to the Apache repository requires password authentication.
Ensure that your work space is cloned from GitHub:
Code Block language bash git clone https://github.com/apache/incubator-trafodion
Ensure that you have a remote pointing to the Apache repository. (Setting only the push URL with user name does not seem to work. This means that fetching the apache remote prompts for a password, but it is ignored. You can hit return without entering password, and fetch still works! You must enter password only when pushing to apache remote.)
Code Block language bash git remote add apache https://USERNAME@git-wip-usUSERNAME@gitbox.apache.org/repos/asf/incubator-trafodion.git
Automated Testing
...
Contributors may notice a review that has several +1’s from other reviewers, passes the functional tests, etc. but the code still has not been merged. As only committers can approve code for merging, you can help things along by getting a committers’s attention in dev@trafodion.incubator.apache.org and letting them know there is a changeset with lots of positive reviews and needs final approval.
...
Be Aware of Licensing Concerns
Trafodion is developed and provided using the Apache 2.0 license. In order to not compromise that license, we must be careful about accepting code contributions that are not original code. We cannot accept code forked from other open source projects that have a non-permissive license for example.
If a reviewer suspects that contributed code is not original without attribution, the reviewer should -1 the change, and in the review comment, state the concern about originality and licensing. If the contributed code is proven to be unoriginal and with an incompatible license, the contribution should be rejected. If this happens, it is recommended that the Trafodion PPMC privately discuss with the contributor the importance of proper licensing and proper attribution or originality. The importance of attribution, originality, and licensing is often difficult for some new contributors, for some students, and for some people from various cultures of origin, and this should be seen as a learning and teaching opportunity.
Merge Pull Requests - The Easy Way
The easy way is to the click the big green button on github that says "Merge Pull Request".
However, in order for committers to have that write-access via github, they must have your apache ID linked to their github ID. Follow directions at: https://gitbox.apache.org/setup/
Merge Pull Requests - The Hard Way
Note | ||
---|---|---|
| ||
Before beginning, make careful note of what branch you are merging to. At the top of the pull request, you'll see words like, "So-and-so wants to merge n commits into apache:target_branch from github_user:some_other_branch. Most of the time, target_branch will be "master", but occasionally you'll see a release branch instead, e.g., "release1.3". In the instructions below, if the target_branch is not "master", replace "master" with the name of the target branch. |
...
Code Block | ||
---|---|---|
| ||
# You will be prompted for your Apache password git push apache HEAD:master`master |
Publish Web Site Updates
If the change you are merging includes website or document updates, you should also update the project website. Look for files in the docs/ tree as well as some files in dcs/ and core/rest that affect documents.
For more info on documentation build, see Modifying Documents.
To publish the website:
...
There are three main types of content on the Apache Trafodion web site:
- The main site content
- Trafodion documents, generated from the Trafodion source code
- Trafodion apidocs, generated from the Trafodion source code and built objects
Depending on which type of content you want to update, you will need to include different steps of the instructions shown below, required steps are marked with an X:
Step | main | docs | apidocs | Description | Command |
---|---|---|---|---|---|
1 | X | X | X | Get the Trafodion source code from git | see "Download and Install Source" |
2 | X | X | Check out a specific release X.Y.Z, if you want to update the documentation for a previous release | git checkout rel/X.Y.Z | |
3 | X | X | Build Trafodion site & documentation, after modifications to the documents, see Modifying Documents. | cd trafodion | |
4 | X | Build Trafodion | see "Build Source" | ||
5 | X | X | X | Get the web site content from git | git clone https://ApacheUserName@gitbox |
...
.apache.org/repos/asf/ |
...
trafodion-site.git |
...
cd trafodion-site |
...
| |||||
6 | X | X | Copy built web/doc changes to main site. This should be done from the master branch (release branch while a release is in progress) ONLY. This should not over-write release-specific docs. | cp -R trafodion/docs/target/* trafodion-site/ | |
7 | X |
...
Copy the release-specific built documentation. This is for non-master branches. | cp -R trafodion |
...
/docs/target |
...
/docs/<release> trafodion-site/docs/ | |||||
8 | X | Build apidocs (note: X.Y.Z is the release number, see step 2 above). TBD: Instructions for copying apidocs to site repository. | cd trafodion-site/apidocs | ||
9 | X | X | X | Commit changes in the web site repository | git status |
10 | X | X | X | Push changes back to ASF repository | git push origin HEAD:asf-site |
11 | X | X | X | Check that the website, https://trafodion.apache.org is refreshed. If it isn't, make a small white-space change to the index.html file to trigger the Apache gitpubsub that does the update. | |
12 | X |
...
Once published, run https://validator.w3.org |
...
against updated pages to ensure that there are no broken links |
...
...
13 | X | X |
...
X | Close the JIRA and/or pull request, if applicable |
...