Skip to content

Added Commitlint and Changelog#18

Open
Symphoria wants to merge 3 commits into
sendilkumarn:masterfrom
Symphoria:enhance/commitlint
Open

Added Commitlint and Changelog#18
Symphoria wants to merge 3 commits into
sendilkumarn:masterfrom
Symphoria:enhance/commitlint

Conversation

@Symphoria

Copy link
Copy Markdown
Contributor

No description provided.

also added Commitizen for easily writing commit messages

ISSUES CLOSED: sendilkumarn#10
@Symphoria Symphoria force-pushed the enhance/commitlint branch from f01b846 to 3509028 Compare March 10, 2018 10:54
@Symphoria Symphoria changed the title Enhance/commitlint Added Commitlint and Changelog Mar 10, 2018
Comment thread CHANGELOG.md Outdated

## New Features

* add circle ci batch ([626fa2f](https://github.com/Symphoria/webpack-addons-pwa/commit/626fa2f))

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you change the repo to point upstream sendilkumarn/webpack-addons-pw?

@Symphoria

Copy link
Copy Markdown
Contributor Author

@sendilkumarn @dhruvdutt Made the changes. Please review

@dhruvdutt dhruvdutt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM except for some minor changes. 👍

Comment thread .cz-config.js Outdated
scopes: [],
types: [
{ value: "feat", name: "feat: A new feature" },
{ value: "tests", name: "tests: Tests, jest, binTestCases, etc" },

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We don't use jest. Also, you can clear binTestCases.

Comment thread .cz-config.js Outdated
{ value: "tests", name: "tests: Tests, jest, binTestCases, etc" },
{ value: "docs", name: "docs: Documentation" },
{ value: "misc", name: "misc: Other formats like tweaks and such" },
{ value: "chore", name: "chore: Updating docs, linting, grunt tasks etc" },

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Clean grunt.

@dhruvdutt dhruvdutt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 💯 @Symphoria Going good.

@sendilkumarn sendilkumarn left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM, few minor changes

Comment thread .eslintignore
@@ -0,0 +1 @@
build No newline at end of file

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

add one more blank line here

@Symphoria Symphoria Mar 11, 2018

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Got it

Comment thread CHANGELOG.md
@@ -0,0 +1,86 @@
<a name="1.0.0"></a>
# 1.0.0 (2018-03-10)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

why it is shown as 1.0.0 ?

@Symphoria Symphoria Mar 11, 2018

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, 1.0.0 is the version specified in package.json. Do I increase the version in package.json? @sendilkumarn

process.stderr.write(`
> Be sure to have the latest git tags locally:
git fetch --tags

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

remove this extra line

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Got it

@sendilkumarn

Copy link
Copy Markdown
Owner

@Symphoria did you plan to work on this ?

@rishabh3112

Copy link
Copy Markdown
Contributor

@Symphoria @sendilkumarn there are few minor changes left in this PR, would be nice if this PR is resolved!!

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.

4 participants