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

clean up 'recursive' parser description #523

Merged
merged 1 commit into from Aug 12, 2023

Conversation

colin-m-davis
Copy link
Contributor

The comment for the recursive parser has a spelling error and its grammar could be improved. This is a very simple pull request that addresses these things!

Thank you for contributing to cats-parse!

This is a kind reminder to run sbt prePR and commit the changed files, if any, before submitting.

@codecov-commenter
Copy link

Codecov Report

Merging #523 (2eb95eb) into main (5c7296b) will increase coverage by 0.13%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

❗ Current head 2eb95eb differs from pull request most recent head 3cfe080. Consider uploading reports for the commit 3cfe080 to get more accurate results

@@            Coverage Diff             @@
##             main     #523      +/-   ##
==========================================
+ Coverage   95.06%   95.20%   +0.13%     
==========================================
  Files          10       10              
  Lines        1480     1480              
  Branches      359      359              
==========================================
+ Hits         1407     1409       +2     
+ Misses         73       71       -2     
Files Changed Coverage Δ
core/shared/src/main/scala/cats/parse/Parser.scala 94.59% <ø> (+0.08%) ⬆️

... and 1 file with indirect coverage changes

@regadas regadas merged commit 4451483 into typelevel:main Aug 12, 2023
13 checks 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.

None yet

3 participants