Skip to content

Conversation

sdangol
Copy link
Contributor

@sdangol sdangol commented Aug 11, 2025

Summary

This PR exports the parse function from the parser package and adds unit tests to check the functionality of the parse function when invoked directly. This was done to be able to do manual parsing if needed

Changes

Please provide a summary of what's being changed

  • Added an export for the parse function
  • Added the same test cases as the tests for the middleware but directly invoking the parse function

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed

Issue number: closes #4263


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@boring-cyborg boring-cyborg bot added parser This item relates to the Parser Utility tests PRs that add or change tests labels Aug 11, 2025
@pull-request-size pull-request-size bot added the size/L PRs between 100-499 LOC label Aug 11, 2025
@sdangol sdangol requested a review from dreamorosi August 11, 2025 12:54
@sdangol sdangol self-assigned this Aug 11, 2025
Copy link

Copy link
Contributor

@dreamorosi dreamorosi left a comment

Choose a reason for hiding this comment

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

Thank you!

@dreamorosi dreamorosi merged commit 4998d6b into main Aug 11, 2025
42 checks passed
@dreamorosi dreamorosi deleted the feat/export-parser branch August 11, 2025 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parser This item relates to the Parser Utility size/L PRs between 100-499 LOC tests PRs that add or change tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: Export the parse function within the parser package
2 participants