Skip to content

Latest commit

 

History

History
96 lines (68 loc) · 6.59 KB

process.md

File metadata and controls

96 lines (68 loc) · 6.59 KB

Using GitHub for comments

Overview

The GEOINT Intelligence Standards Center of Excellence intends to experiment with the use of GitHub as a collaboration platform, and the document formats suppored by GitHub as a way to evolve the specification editing and public comment process. The versions of draft specifications in GitHub, in whole or in part, should not be considered official positions of the NGA as an organization. Official specifications will be published on the DoD Information Technology Standards Registry (DISR).

Comments

Comments on the specifications can be made in one of two ways:

  1. Pull Requests (PRs) that contain the proposed changes to the document, with a comment from the author. These should have a label for the priority. (preferred)
  • Pull Requests be made for each logical set of changes to the spec, against a user's branch, not their master. Pull Requests that do too many things at once will be rejected and the submitter will be asked to break it up in to multiple pull requests.
  • Submitters should only change their pull requests in dialog with the GWG FG Chair/POC. In other words after submitting they should not modify their request at all unless asked to by the FG Chair.
  1. If the comment is of a more general nature then create a new 'issue' in the standard GitHub repo issue tracker, with proper labeling as well as a comment on what section of the spec it applies to.

You can review and comment on submitted pull requests and issues

No need to assign a milestone, the FG Chair/POC will reviews PR and issues, and assign them to the appropriate milestone if they are valid.

If accepted the FG Chair will accept the change or provide a reason as to why it is not incorporated in the ticket, and then close it.

Using GitHub

For those who are new to GitHub the following information may help get you up to speed.

GUI only Editing

Most everything needed to edit the GeoPackage repository can be done completely through the web with GitHub, no need to learn git.

You can just hit 'edit' on any of the specification pages. This will automatically 'fork' the repository in to your own copy. You can also create a fork before editing by hitting the 'Fork' button at the top right of this page. This creates a personal copy of the geopackage specification repository (a fork). You can find more detailed instructions on forking a repo on the Fork A Repo page.

To edit the document we are using Markdown, with GitHub's extensions. For quick hints on markdown this cheat sheet can be quite helpful.

After editing there you can create a 'pull request' from your fork to submit the work for review and merging. You just hit 'compare and review' from your own version of the repository, or any branch in it, and then can click to create a pull request right there.

If you are making a more substantial set of changes, or more than one logical change to the spec, it is best to create a branch to work on the set. Pull requests are associated with the branches they are made on, so if you make more changes based on feedback then all changes can get pulled in when ready.

Note you can create new branches as well as new files completely through the web. You can even create new directories:

creating directories

If there is anything that you need to do with git, but you aren't comfortable with using it and don't want to learn it, then just create an issue and someone will certainly be able to assist.

Using Git

There are countless tutorials on using Git and Github, but we will give a quick overview here and point to more resources.

For instance, after creating your fork of the geopackage repository you will need to copy it to your computer (clone). The URL of your fork will be https://github.com/username/geopackage.git where username should be replaced by your github username. Cloning the repository can be done using the git command-line client (git clone https://github.com/username/geopackage.git) or using a git GUI like the GitHub Windows or Mac or SourceTree.

Once the repository has been cloned to your computer you can make the changes to the specification that you would like to suggest. Many text editors have plugins for formatting and previewing Markdown.

Once you're done making your changes, commit them and push them to the github servers. The free Pro Git book provides detailed instructions on committing changes and pushing changes to a server.

Now that your change has been pushed to github you should initiate a pull request to request that your change be integrated in the master copy of the specification. The GitHub pull request help page provides detailed instructions on how to do this.

Process

  • FG Chair assigns pull requests/tickets to volunteer FG members to work on resolution of the comment.
    • the volunteer becomes the 'owner', and is expected to move it towards a vote.
    • The owner gets any needed discussion going.
    • A proposed resolution is written in the comments of the ticket by the owner (could just be 'vote we accept this', or could be 'propose to reject this because of x,y,z reasons)
    • Owner marks it as 'ready for vote' and marks the milestone for the next teleconference.
    • ideally dialog happens in the pull requests about the merits, and people +1 in the comments in support of it.
  • FG teleconferences review tickets.
    • Tickets with lots of comments that aren't resolved are discussed in the meetings. Ideally with an owner assigned to change the PR to reflect the consensus of the group.
    • Tickets with lots of +1's should be moved through quickly, basically most all the 'homework' should be done before the teleconference, telecons are just used to discuss open issues and ratify the +1's in the ticket.
    • An 'editor' either merges the pull request or post the official rejection comment, and closes the ticket.