Skip to content

Conversation

@nicoledow
Copy link
Contributor

@nicoledow nicoledow commented Dec 4, 2023

Items Addressed

Addresses issue #134 by adding a table to the ReadMe that outlines the options parsers currently accept.

Note that #132 adds an option to the BooleanParser. Depending on the order that this PR and #132 are merged, one or the other will need to update the table with the new option.

Review Questions

  • I did not update the version number since this is just an update to the ReadME. Should I have?

Author Checklist

  • Add unit test(s)
  • Update documentation (if necessary)
  • Update version in version.rb following versioning guidelines

@nicoledow nicoledow marked this pull request as ready for review December 4, 2023 16:24
@nicoledow nicoledow requested a review from chawes13 as a code owner December 4, 2023 16:24
@nicoledow nicoledow changed the title Enhance ReadMe with documentation on parser options 134 Enhance ReadMe with documentation on parser options Dec 7, 2023
Copy link
Contributor

@chawes13 chawes13 left a comment

Choose a reason for hiding this comment

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

LGTM! Can you add bump the version by a patch?

@nicoledow
Copy link
Contributor Author

LGTM! Can you add bump the version by a patch?

Done!

@nicoledow nicoledow requested a review from chawes13 January 5, 2024 15:49
@chawes13 chawes13 merged commit 682539d into main Jan 5, 2024
@chawes13 chawes13 deleted the 134-enhance-readme-with-documentation-on-parser-options-for-default-parsers branch January 5, 2024 16:08
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.

Enhance README with documentation on parser options for default parsers

3 participants