Skip to content

Conversation

dimabory
Copy link
Contributor

PR Checklist

Overview

  • Added filtering of TSLint rules arguments to allow only some of them, which are available in ESLint.
  • Added test.

Copy link
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

Good start! 💖 A little bit extra is needed around notices if you don't mind!

@JoshuaKGoldberg JoshuaKGoldberg added the status: waiting for author The PR author should address requested changes label Dec 12, 2019
Copy link
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

This is looking good! I left a couple of comments @dimabory if you have time to address them. If you don't soon, that's fine too! We can merge this in and file those as follow-up issues. 🚀

@JoshuaKGoldberg JoshuaKGoldberg added status: waiting for reviewer Waiting for a maintainer to review and removed status: waiting for author The PR author should address requested changes labels Dec 17, 2019
Copy link
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

Great stuff, thanks again @dimabory !

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

Labels

status: waiting for reviewer Waiting for a maintainer to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrectly generated space-before-function-paren

2 participants