Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CONTRIBUTING: Proposal #90

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

CONTRIBUTING: Proposal #90

wants to merge 1 commit into from

Conversation

Kreyren
Copy link
Contributor

@Kreyren Kreyren commented Jul 26, 2019

Proposal to improve the efficiency of development on oh-my-bash project
Fixes: #89
Fixes: #88
Fixes: #86
Bug: #85 - Partially fixes
Bug: #83 - Partially fixes
Signed-off-by: Jacob Hrbek [email protected]

Proposal to improve the efficiency of development on oh-my-bash project
Fixes: ohmybash#89
Fixes: ohmybash#88
Fixes: ohmybash#86
Partially Fixes: ohmybash#85
Partially Fixes: ohmybash#83
Signed-off-by: Jacob Hrbek <[email protected]>
nntoan
nntoan previously approved these changes Aug 14, 2019
@nntoan nntoan dismissed their stale review August 14, 2019 04:51

Standardize the commit message

@Kreyren
Copy link
Contributor Author

Kreyren commented Aug 14, 2019

@nntoan You dismissed it, but commented in a way that seems like approval, please clarify.

@Kreyren Kreyren self-assigned this Aug 15, 2019
@Kreyren Kreyren added P3 - Normal Priority 3 - Normal P2 - Important Priority 2 and removed P3 - Normal Priority 3 - Normal labels Aug 15, 2019
@Kreyren
Copy link
Contributor Author

Kreyren commented Aug 15, 2019

Assigned P2 since this is maintainance improvement issue.

@nntoan
Copy link
Member

nntoan commented Aug 20, 2019

@nntoan You dismissed it, but commented in a way that seems like approval, please clarify.

@Kreyren When I clicked to Approve button I forgot to hit the Submit on the #86 , that's why I have to dismiss the review then finished the proposal over there.

@Kreyren
Copy link
Contributor Author

Kreyren commented Aug 20, 2019

@nntoan You dismissed it, but commented in a way that seems like approval, please clarify.

@Kreyren When I clicked to Approve button I forgot to hit the Submit on the #86 , that's why I have to dismiss the review then finished the proposal over there.

noted


Oh-My-Bash is a community-driven project. Contribution is welcome, encouraged and appreciated.
It is also essential for the development of the project.
Oh-My-Bash is Free and Open Source project under the terms of GNU General Public License v2.0 relying on contributions from third parties.
Copy link
Member

Choose a reason for hiding this comment

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

@Kreyren Can you please update to a new contributing proposal to match with our current License?
Thanks.

Copy link
Contributor

Choose a reason for hiding this comment

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

Now we are licensing OMB with the MIT license.

You can also take a look at the [FAQ](https://github.com/ohmybash/oh-my-bash/wiki/FAQ)
to be sure your contribution has not already come up.
### Quality Assurance (QA)
All merge requests have to pass spellcheck (https://www.shellcheck.net/) unless stated otherwise.
Copy link
Contributor

Choose a reason for hiding this comment

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

typo spellcheck -> shellcheck

Suggested change
All merge requests have to pass spellcheck (https://www.shellcheck.net/) unless stated otherwise.
All merge requests have to pass shellcheck (https://www.shellcheck.net/) unless stated otherwise.

The same typos are also found in other lines.


Oh-My-Bash is a community-driven project. Contribution is welcome, encouraged and appreciated.
It is also essential for the development of the project.
Oh-My-Bash is Free and Open Source project under the terms of GNU General Public License v2.0 relying on contributions from third parties.
Copy link
Contributor

Choose a reason for hiding this comment

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

Now we are licensing OMB with the MIT license.


### Getting started

## Merge Requests
Copy link
Contributor

Choose a reason for hiding this comment

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

As this is GitHub (but not GitLab or other services), it is more natural to call it Pull Requests. Any thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 - Important Priority 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

REQUEST: DevOps QA: Commit messages REQUEST: Write access
3 participants