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.

Sign up for GitHub

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

Jump to bottom

feat: update config and dependencies #8

Merged
merged 1 commit into from Sep 10, 2020
Merged

feat: update config and dependencies #8

merged 1 commit into from Sep 10, 2020

Conversation

LexSwed
Copy link
Contributor

@LexSwed LexSwed commented May 13, 2020 •
edited

Proposal

This PR:

otorrillas
otorrillas approved these changes May 14, 2020
View reviewed changes
Copy link
Contributor

@otorrillas otorrillas left a comment

Choose a reason for hiding this comment

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

Looks good to me, much simpler.

Are there breaking changes?

package.json Outdated Show resolved Hide resolved
@LexSwed
Copy link
Contributor Author

LexSwed commented May 21, 2020

Available via yarn add -D @typeform/eslint-config@next

@LexSwed
feat!: use react-scripts rules
dbf0727 
BREAKING CHANGE: new rules set
@LexSwed LexSwed merged commit a79cbcc into master Sep 10, 2020
@LexSwed LexSwed deleted the next branch September 10, 2020 11:20
@travis-tf
Copy link

🎉 This PR is included in version 2.0.0 🎉

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
Reviewers

@otorrillas otorrillas otorrillas approved these changes

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

no-unused-expressions makes an error for optional chaining of method calls: foo?.bar();
3 participants
@LexSwed @travis-tf @otorrillas