Skip to content
This repository was archived by the owner on Aug 30, 2021. It is now read-only.

Update development.js#1859

Closed
ttawfic wants to merge 1 commit into
meanjs:masterfrom
ttawfic:patch-1
Closed

Update development.js#1859
ttawfic wants to merge 1 commit into
meanjs:masterfrom
ttawfic:patch-1

Conversation

@ttawfic
Copy link
Copy Markdown

@ttawfic ttawfic commented Aug 29, 2017

Commit Message Guidelines

Header
Blank Line
Body
Blank Line
Footer

The header should look like:
<type>(<scope>): <subject>

The body should have any necessary detailed info about the commit:
An example, references as to where this idea came from, etc.

The footer should have all the issues tagged:
Fixes #123, Fixes #456

So a commit should look like:
feat(users): Add new Yahoo authentication

Yahoo authentication idea proposed by @codydaig
Example implementation in file.js

Fixes #82
  • Types:
    • feat - Features, Enhancements, and overall Improvements
    • fix - Fixes, Bugs, HotFixs, etc...
    • doc - Changes to the Documentation and doesn't actually touch any code.
  • Scope:
    • The scope should be where the change took place.
    • Examples: users, core, config, articles
  • Subject:
    • The subject line should be clear and concise as to what is being accomplished in the commit.
  • General Rules:
    • No Line in the Commit message can be longer than 80 characters.
  • Reference: Angular Conventions

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.

2 participants