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

Update README.md #131

Merged
merged 3 commits into from
Nov 30, 2023
Merged

Update README.md #131

merged 3 commits into from
Nov 30, 2023

Conversation

ollllla
Copy link
Contributor

@ollllla ollllla commented Nov 24, 2023

My team is developing a modern multifunctional tool for processing the Jest tests, please add it to the awesome testing list, we'll be grateful!

My team is developing a modern multifunctional tool for processing the Jest tests, please add it to the awesome testing list, we'll be grateful!
@mrazauskas
Copy link
Contributor

Thanks!

Could you move the entries to the bottom, please? See, the Contribution Guidelines:

- Additions should be added to the bottom of the relevant category.


Perhaps it would be enough to have only one entry? If I get it right, you are pointing to different parts of documentation to make different features of the tool visible. Not sure if that is necessary.

I made the fix according to your comment.

Yes, you correctly noticed that the basis of Gihhub Pipe is the same reporter, but the tools are another for a different audience. It does not require sign-in in TMS and its token.
Copy link
Contributor Author

@ollllla ollllla left a comment

Choose a reason for hiding this comment

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

I made the fix according to your comment.

Yes, you correctly noticed that the basis of Gihhub Pipe is the same reporter, but the tools are another for a different audience. It does not require sign-in in TMS and its token.

@mrazauskas
Copy link
Contributor

Not sure about it. Let's see what others think.

@SimenB
Copy link
Member

SimenB commented Nov 30, 2023

Yeah, I think only the jest specific link is needed

Copy link
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

thanks!

@SimenB SimenB merged commit 35f7dd3 into jest-community:main Nov 30, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants