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

refactor README #869

Merged
merged 2 commits into from
May 25, 2022
Merged

Conversation

connectdotz
Copy link
Collaborator

first draft to add a quick-start section and refactor the rest README.

address #863

@coveralls
Copy link

coveralls commented May 23, 2022

Pull Request Test Coverage Report for Build 2373572131

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.369%

Totals Coverage Status
Change from base Build 2255224901: 0.0%
Covered Lines: 2569
Relevant Lines: 2640

💛 - Coveralls

@connectdotz
Copy link
Collaborator Author

@isidorn I end up refactoring more than I initially intended to. Feel free to focus on the "Getting Started" section and let me know if it will work for the test doc.

@isidorn
Copy link

isidorn commented May 23, 2022

@connectdotz thanks. I think this looks good 👏
Do you also want to contribute the same section on our side? Or would you prefer that I do it? Thanks!

@connectdotz
Copy link
Collaborator Author

@isidorn I am happy to, but I think it might be better if you do it, that way the style and tone will be more consistent across sections.

We will keep an eye on the doc if changes are needed in the future.

@connectdotz connectdotz merged commit 1432104 into jest-community:master May 25, 2022
@isidorn
Copy link

isidorn commented May 30, 2022

@connectdotz makes sense, I will look into this. Thanks

legend1202 pushed a commit to legend1202/vscode-jest that referenced this pull request Jun 18, 2023
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