Skip to content

6236 – Deprecate parsers#409

Merged
vasconsaurus merged 1 commit intodevelopfrom
6236-deprecate-parsers
Mar 5, 2025
Merged

6236 – Deprecate parsers#409
vasconsaurus merged 1 commit intodevelopfrom
6236-deprecate-parsers

Conversation

@vasconsaurus
Copy link
Contributor

Description

It was confirmed by program that these two parsers are no longer used.

References: CV2-6236

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can verify the changes. Please describe whether or not you implemented automated tests.

Things to pay attention to during code review

Please describe parts of the change that require extra attention during code review, for example:

  • File FFFF, line LL: This refactoring does this and this. Is it consistent with how it’s implemented elsewhere?
  • Etc.

Checklist

  • I have performed a self-review of my own code
  • I have added unit and feature tests, if the PR implements a new feature or otherwise would benefit from additional testing
  • I have added regression tests, if the PR fixes a bug
  • I have added logging, exception reporting, and custom tracing with any additional information required for debugging
  • I considered secure coding practices when writing this code. Any security concerns are noted above.
  • I have commented my code in hard-to-understand areas, if any
  • I have made needed changes to the README
  • My changes generate no new warnings

@vasconsaurus vasconsaurus force-pushed the 6236-deprecate-parsers branch from 5e49ea3 to 7cc9041 Compare March 5, 2025 17:40
@vasconsaurus vasconsaurus requested a review from caiosba March 5, 2025 17:47
@vasconsaurus vasconsaurus marked this pull request as ready for review March 5, 2025 17:49
@vasconsaurus vasconsaurus merged commit 01a280e into develop Mar 5, 2025
4 checks passed
@vasconsaurus vasconsaurus deleted the 6236-deprecate-parsers branch March 5, 2025 17:49
vasconsaurus added a commit that referenced this pull request Mar 5, 2025
It was confirmed by program that these two parsers are no longer used.

References: CV2-6236
@vasconsaurus vasconsaurus mentioned this pull request Oct 24, 2025
8 tasks
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.

2 participants