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

Update version check in validate_spec to support 1.X.Y version. #1793

Merged
merged 2 commits into from
Apr 9, 2023

Conversation

utam0k
Copy link
Member

@utam0k utam0k commented Apr 9, 2023

Previously, the validate_spec function only supported runtime spec versions that started with "1.0".
This commit updates the function to support all versions starting with "1." (e.g., 1.X.Y), making it compatible with a broader range of runtime spec versions.

containerd/runwasi#78 (comment)

Previously, the validate_spec function only supported runtime spec versions that started with "1.0".
This commit updates the function to support all versions starting with "1." (e.g., 1.X.Y), making it compatible with a broader range of runtime spec versions.

Signed-off-by: utam0k <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Apr 9, 2023

Codecov Report

Merging #1793 (7104494) into main (cb1340f) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1793      +/-   ##
==========================================
- Coverage   68.64%   68.63%   -0.01%     
==========================================
  Files         121      121              
  Lines       13324    13325       +1     
==========================================
  Hits         9146     9146              
- Misses       4178     4179       +1     

@utam0k utam0k requested a review from a team April 9, 2023 05:41
@utam0k
Copy link
Member Author

utam0k commented Apr 9, 2023

@yihuaf Thanks for your review 🙇

@utam0k utam0k merged commit edd63c8 into youki-dev:main Apr 9, 2023
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.

3 participants