Skip to content
This repository has been archived by the owner on Oct 16, 2023. It is now read-only.

Congratulate and introduce new users in general channel #166

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

Conversation

ya7on
Copy link
Contributor

@ya7on ya7on commented Feb 7, 2019

close #162

Please make sure all below checkboxes in the Mandatory section have been checked before submitting your PR (also don't forget to pay attention to the Depending on Circumstances section)

Mandatory

  • The commit message is an imperative and starts with a capital letter (for example, Add something, Remove something, Fix something, etc.).
  • I made sure I hadn't put a period at the end of the the commit message(s).

Depending on Circumstances

Some of the items in the section become mandatory if you are a first-time contributor and/or your changes are relevant to the items.

  • Adding a new parameter I didn't forget to reflect it in README.md.
  • Modifying a list of constants or imports I didn't brake an alphabetical order.
  • (for first-time contributors) One of the commits in the pull request adds me to the Acknowledgements section in AUTHORS.md.

@ya7on ya7on requested a review from a team February 7, 2019 12:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Congratulate and introduce new users in general channel
1 participant