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.

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

Added a link to the deprecations warning to point to a codemod to help address it #140

Merged
merged 1 commit into from
Aug 30, 2017

Conversation

lorcan
Copy link
Contributor

@lorcan lorcan commented Aug 25, 2017

Hi folks, I wrote up a codemod to help address the testSelector deprecation introduced in #134.

The codemod is here: https://github.com/lorcan/test-selectors-codemod

I thought it might be helpful to add a link to the codemod to the deprecation warning in the hopes that it makes it a little easier to fix :-)

@Turbo87
Copy link
Collaborator

Turbo87 commented Aug 26, 2017

hmm, I think I'd prefer it if we'd put it in the readme and use the URL property of the deprecation to link to the readme section

@Turbo87
Copy link
Collaborator

Turbo87 commented Aug 26, 2017

forgot to mention: nice work!!

@lorcan lorcan force-pushed the lorcan/add-link-to-codemod branch from acd747a to 4bcc432 Compare August 26, 2017 09:01
@lorcan
Copy link
Contributor Author

lorcan commented Aug 26, 2017

Great, thanks, I updated the PR. Hope the copy is ok, happy to tweak it 👍

Copy link
Collaborator

@Turbo87 Turbo87 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, thanks :)

@lorcan
Copy link
Contributor Author

lorcan commented Aug 29, 2017

Cool, is there left to do to get this merged? :-)

@marcoow marcoow merged commit a521f69 into mainmatter:master Aug 30, 2017
@lorcan lorcan deleted the lorcan/add-link-to-codemod branch August 30, 2017 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants