Skip to content

Integrate with @comicrelief/eslint-config (#83)#94

Merged
dipasqualew merged 1 commit into
masterfrom
83-integrate-with-comicrelief-eslint-config
Aug 27, 2020
Merged

Integrate with @comicrelief/eslint-config (#83)#94
dipasqualew merged 1 commit into
masterfrom
83-integrate-with-comicrelief-eslint-config

Conversation

@dipasqualew
Copy link
Copy Markdown
Contributor

@dipasqualew dipasqualew commented Aug 25, 2020

In order to not introduce regression on highly complex functions, some of the rules have been disabled, with the aim of resolving disabled rules one by one in future releases.

Added the yarn lint command.

Closes #83

@dipasqualew dipasqualew force-pushed the 83-integrate-with-comicrelief-eslint-config branch from b1d7ab0 to 9ac7758 Compare August 25, 2020 12:20
@dipasqualew dipasqualew changed the title Integrated with @comicrelief/eslint-config (#83) Integrate with @comicrelief/eslint-config (#83) Aug 25, 2020
@dipasqualew dipasqualew self-assigned this Aug 26, 2020
Comment thread .eslintrc Outdated
@dipasqualew dipasqualew requested a review from lebaz20 August 27, 2020 09:07
Copy link
Copy Markdown
Contributor

@lebaz20 lebaz20 left a comment

Choose a reason for hiding this comment

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

Commit messages need to trigger semantic-release process.

@lebaz20
Copy link
Copy Markdown
Contributor

lebaz20 commented Aug 27, 2020

@dipasqualew you can edit commit messages to include fix: for instance and push force may be?

@dipasqualew
Copy link
Copy Markdown
Contributor Author

I don't think it's required because this change doesn't provide any new feature nor change in functionality. It's merely a dev update, so I don't think there's anything to release

@dipasqualew dipasqualew requested a review from lebaz20 August 27, 2020 11:46
lebaz20
lebaz20 previously approved these changes Aug 27, 2020
In order to not introduce regression on highly complex
functions, some of the rules have been disabled, with the aim
of resolving disabled rules one by one in future releases.

Added the `yarn lint` command.
@dipasqualew dipasqualew force-pushed the 83-integrate-with-comicrelief-eslint-config branch from 684c731 to 03fa011 Compare August 27, 2020 11:56
@dipasqualew dipasqualew merged commit 0f71a2a into master Aug 27, 2020
@dipasqualew dipasqualew deleted the 83-integrate-with-comicrelief-eslint-config branch August 27, 2020 14:28
@lebaz20
Copy link
Copy Markdown
Contributor

lebaz20 commented Aug 27, 2020

🎉 This PR is included in version 1.1.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate with @comicrelief/eslint-config.

2 participants