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

Emphasizing the Issue and PR process #240

Merged
merged 9 commits into from
Jul 3, 2014

Conversation

Martii
Copy link
Member

@Martii Martii commented Jul 2, 2014

Applies to #60 and #167

@sizzlemctwizzle
Would you please skim over these additions and see if I missed anything or want something changed? I do have some grammar issues which are good to correct for translators... and I'll get to that in a bit.

I also have no idea what the upstream label means which is blocking the PR READY label for this... what is it?

@Martii Martii self-assigned this Jul 2, 2014
@Martii Martii removed their assignment Jul 2, 2014
@Martii Martii added the question label Jul 3, 2014

The following is a brief list of **some** of the labels used on the project and is used to establish teamwork. Not everyone has permission to set these and usually will be set by someone unless expressly prohibited:

##### BLOCKING
Copy link
Member

Choose a reason for hiding this comment

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

"This label means that merging unrelated or non-bugfix PRs will be put on hold until this label has been removed."

@sizzlemctwizzle
Copy link
Member

I also have no idea what the upstream label means

No clue. Did I add that? Anyway, I just deleted it.

see if I missed anything

Aside from my line comments, it looks like you got everything.

* More descript elaboration
* Some spelling corrections.. ran it through a spell checker.
* Removed `upstream` label ref and question
* Changed `console` to `terminal` since console is more KDE oriented and terminal is more X oriented. e.g. devoid of Window Manager.

Applies to OpenUserJS#240, OpenUserJS#167 and OpenUserJS#60
@Martii Martii added the PR READY label Jul 3, 2014
sizzlemctwizzle added a commit that referenced this pull request Jul 3, 2014
Emphasizing the Issue and PR process
@sizzlemctwizzle sizzlemctwizzle merged commit 5579995 into OpenUserJS:master Jul 3, 2014
@Martii Martii deleted the CONTRIBUTING branch July 7, 2014 09:32
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DOC Pertains inclusively to the Documentation operations.
Development

Successfully merging this pull request may close these issues.

2 participants