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

refactor: validation logic for latest version in metadata.json #463

Merged

Conversation

scriptnull
Copy link
Member

This is a refactor of things from #460

I am opening this to convey some of the code review points that I had in mind (which was a bit hard to convey in the original PR).

I suggest doing the below before making use of the code from this PR:

  • Add tests wherever possible
  • Try out the new validate logic (I haven't run it in local yet)
  • Feel free to ask me questions on why I have written something in a certain way (compared to the original PR)

@codingkarthik codingkarthik merged commit 0aac17a into kc/validate-latest-version Feb 17, 2025
@codingkarthik codingkarthik deleted the refactor/kc/validate-latest-version branch February 17, 2025 06:27
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