Skip to content

Wrong link to docs in README #74

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

Merged
merged 1 commit into from
Feb 3, 2020
Merged

Wrong link to docs in README #74

merged 1 commit into from
Feb 3, 2020

Conversation

arvigeus
Copy link
Contributor

@arvigeus arvigeus commented Feb 1, 2020

Missing .md extension

Missing .md extension
@arvigeus
Copy link
Contributor Author

arvigeus commented Feb 1, 2020

Another one I found: prefer-explicit-assert suggests using it in conjunction with prefer-expect-query-by, which seems not existing.

@Belco90
Copy link
Member

Belco90 commented Feb 1, 2020

Another one I found: prefer-explicit-assert suggests using it in conjunction with prefer-expect-query-by, which seems not existing.

Ah we forgot to remove that. Can you address that change too? Thanks!

Copy link
Member

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the missed extension! If you could remove old reference to non-existing rule would be nice.

@Belco90 Belco90 merged commit 82527f8 into testing-library:master Feb 3, 2020
@Belco90
Copy link
Member

Belco90 commented Feb 3, 2020

We can address the suggestion for non-existing rule in a different PR.

@all-contributors please add @arvigeus for docs

@allcontributors
Copy link
Contributor

@Belco90

I've put up a pull request to add @arvigeus! 🎉

@arvigeus arvigeus deleted the patch-1 branch February 3, 2020 10:40
@arvigeus
Copy link
Contributor Author

arvigeus commented Feb 3, 2020

I'll remove the rule later today. Thanks!

@Belco90
Copy link
Member

Belco90 commented Feb 3, 2020

Don't worry! Addressed already in #78

Thanks for catching this!

@Belco90
Copy link
Member

Belco90 commented Feb 4, 2020

🎉 This PR is included in version 2.2.1 🎉

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.

2 participants