Skip to main content

Notice: this Wiki will be going read only early in 2024 and edits will no longer be possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.

Jump to: navigation, search

Difference between revisions of "EDT:How to commit code to EDT"

(File encoding)
(Builds)
 
(5 intermediate revisions by the same user not shown)
Line 1: Line 1:
== Bugzilla bugs are mandatory ==
+
== Bugzilla bugs are mandatory ==
  
Every change we commit <span style="color:#FF0000">'''''MUST'''''</span> have a corresponding bug or enhancement in Bugzilla. Be sure the Target Milestone is set correctly.  
+
Every change we commit <span style="color:#FF0000">'''''MUST'''''</span> have a corresponding bug or enhancement in Bugzilla. Be sure to set the bug's Target Milestone correctly.  
  
When you resolve a bug, you <span style="color:#FF0000">'''''MUST'''''</span> list the files that you changed.  Either type in the file names or attach a screenshot that shows the changes.
+
== Git commit messages are mandatory ==
  
== Git commit messages are mandatory  ==
+
You <span style="color:#FF0000">'''''MUST'''''</span> enter a message when you commit a change to Git. The message <span style="color:#FF0000">'''''MUST'''''</span> include the number of the Bugzilla bug or enhancement.
  
You <span style="color:#FF0000">'''''MUST'''''</span> enter a message when you commit a change to Git. The message <span style="color:#FF0000">'''''MUST'''''</span> include number of the Bugzilla bug or enhancement.
+
Commit messages should be concise. It's best to say ''why'' you made the change, as opposed to saying ''what'' you changed.  
 
+
Commit messages should be concise. It's best to say why you made the change, as opposed to saying what you changed.  
+
  
 
*Bad: "Bug 12345 - Fixed."  
 
*Bad: "Bug 12345 - Fixed."  
Line 15: Line 13:
 
*Best: "Bug 12345 - The input to the gas pump must be in gallons, not quarts."
 
*Best: "Bug 12345 - The input to the gas pump must be in gallons, not quarts."
  
== File encoding ==
+
If you want to write more about your changes, do it in Bugzilla.
 +
 
 +
== Links between bugs and fixes are mandatory ==
 +
 
 +
When you resolve a bug, you <span style="color:#FF0000">'''''MUST'''''</span> include the ID of your fix's Git commit. If there's more than one commit, include all of their IDs.
 +
 
 +
The easiest way to find the ID of a commit is with the Git Reflog view, which lists the commits in a particular branch. Double-click on your commit to open it in the Commit Viewer. Right-click on the ID at the top of the Commit Viewer and choose Copy SHA-1.
 +
 
 +
[[Image:copySHA1.png]]
 +
 
 +
Now you can paste the commit ID into Bugzilla.
 +
 
 +
== File encoding ==
  
 
We use the UTF-8 encoding for all of our files. Set your workspace's Text File Encoding preference to UTF-8. The preference is on the General > Workspace preference page.
 
We use the UTF-8 encoding for all of our files. Set your workspace's Text File Encoding preference to UTF-8. The preference is on the General > Workspace preference page.
  
== Timing  ==
+
== Builds ==
 +
 
 +
Our Nightly builds run twice a day, at 9 AM and 9 PM Eastern U.S. time. If no changes were made since the last build, then a new build is not produced.
  
Our Nightly builds run twice a day: at 9 AM and 9 PM Eastern U.S. time.  
+
Builds for milestones and releases run as needed.
  
== Which files are included in the build ==
+
== Which files are included in the build ==
  
Our builds compile the latest code from the ''master'' branch. There are also a few projects which are not compiled, but are used during the build. For example, we have a set of projects which contain the .egl and .js files for our widgets. The RUI widget libraries are created from them.
+
Our builds compile the latest code from the ''master'' branch. There are also a few projects which are not compiled, but are used during the build. For example, we have a set of projects which contain the .egl and .js files for our widgets. The build creates the RUI widget libraries from them.
  
 
[[Category:EDT]]
 
[[Category:EDT]]

Latest revision as of 08:47, 21 August 2012

Bugzilla bugs are mandatory

Every change we commit MUST have a corresponding bug or enhancement in Bugzilla. Be sure to set the bug's Target Milestone correctly.

Git commit messages are mandatory

You MUST enter a message when you commit a change to Git. The message MUST include the number of the Bugzilla bug or enhancement.

Commit messages should be concise. It's best to say why you made the change, as opposed to saying what you changed.

  • Bad: "Bug 12345 - Fixed."
  • Better: "Bug 12345 - Added quartsToGallons() on calls to pumpGas()."
  • Best: "Bug 12345 - The input to the gas pump must be in gallons, not quarts."

If you want to write more about your changes, do it in Bugzilla.

Links between bugs and fixes are mandatory

When you resolve a bug, you MUST include the ID of your fix's Git commit. If there's more than one commit, include all of their IDs.

The easiest way to find the ID of a commit is with the Git Reflog view, which lists the commits in a particular branch. Double-click on your commit to open it in the Commit Viewer. Right-click on the ID at the top of the Commit Viewer and choose Copy SHA-1.

CopySHA1.png

Now you can paste the commit ID into Bugzilla.

File encoding

We use the UTF-8 encoding for all of our files. Set your workspace's Text File Encoding preference to UTF-8. The preference is on the General > Workspace preference page.

Builds

Our Nightly builds run twice a day, at 9 AM and 9 PM Eastern U.S. time. If no changes were made since the last build, then a new build is not produced.

Builds for milestones and releases run as needed.

Which files are included in the build

Our builds compile the latest code from the master branch. There are also a few projects which are not compiled, but are used during the build. For example, we have a set of projects which contain the .egl and .js files for our widgets. The build creates the RUI widget libraries from them.

Back to the top