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

Triangulate backward compatibility check commands #1257

Merged
merged 18 commits into from
Oct 28, 2024

Conversation

yogeshnikam671
Copy link
Member

@yogeshnikam671 yogeshnikam671 commented Aug 21, 2024

What:
Created an abstract class which has the common logic around backward compatibility check. This helps reuse the logic in other protocols easily.

Checklist:

  • Documentation
  • Tests
  • Tested locally
  • Sonar Quality Gate

@yogeshnikam671 yogeshnikam671 changed the title Triangulate bcc commands Triangulate backward compatibility check commands Aug 21, 2024
@yogeshnikam671 yogeshnikam671 force-pushed the triangulate_bcc_commands branch from 17ec532 to 7962cc1 Compare August 22, 2024 06:22
@yogeshnikam671 yogeshnikam671 force-pushed the triangulate_bcc_commands branch from 9f6610e to 69dcc3d Compare August 26, 2024 07:13
@yogeshnikam671 yogeshnikam671 force-pushed the triangulate_bcc_commands branch 3 times, most recently from 59887e6 to bd50941 Compare August 30, 2024 12:06
@yogeshnikam671 yogeshnikam671 force-pushed the triangulate_bcc_commands branch from bd50941 to 63e0403 Compare September 27, 2024 11:17
@yogeshnikam671 yogeshnikam671 force-pushed the triangulate_bcc_commands branch 2 times, most recently from b6fbbce to 827a305 Compare October 10, 2024 05:01
@yogeshnikam671 yogeshnikam671 force-pushed the triangulate_bcc_commands branch from dd5b2b5 to 9b37ba7 Compare October 14, 2024 15:17
Abstract out the common logic in backward-compatibility-check command
…ommand which can let us run the check on a specific folder or file in a repository
Add deprecation notice to all the b/w compatibility related commands which will be eventually removed
…l bring the repository back to the original state if the command is aborted in between
…d unnecessary logs in the backward compatibility check result logs
…le override in sub-classes of BackwardCompatibilityCheckBaseCommand
@yogeshnikam671 yogeshnikam671 force-pushed the triangulate_bcc_commands branch from 9b37ba7 to 444af8d Compare October 28, 2024 12:22
Copy link
Member

@harikrishnan83 harikrishnan83 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@harikrishnan83 harikrishnan83 merged commit b35abe9 into main Oct 28, 2024
2 checks passed
@harikrishnan83 harikrishnan83 deleted the triangulate_bcc_commands branch October 28, 2024 12:51
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