Skip to content

Conversation

@amarts
Copy link
Owner

@amarts amarts commented Aug 12, 2020

Change-Id: I31becd7cdb35cb68048b662786c31694791d50fe
Signed-off-by: Amar Tumballi amar@kadalu.io

CONTRIBUTING.md Outdated
## Development Workflow

0. Fork Repository
- Visit [GlusterFS repository](https://github.com/gluster/glusterfs) and click 'Fork'.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

- Commit changes as a new commit on top of the original commits in the branch, and push the changes to same branch (issueNNNN)
- Commit changes into the same commit with `--amend` option, and do a push to the same branch with `--force` option.

6. Merging:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge type is "Squash and Merge"?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Only that is permitted. Will update the document

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

* Make CONTRIBUTING.md have complete workflow documented.
* Change './rfc.sh' to reflect the new workflow

Updates: gluster/project-infrastructure#62
Change-Id: I31becd7cdb35cb68048b662786c31694791d50fe
Signed-off-by: Amar Tumballi <amar@kadalu.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants