Skip to content

fix: Parse ubuntu suffix#1982

Merged
hogo6002 merged 5 commits intogoogle:mainfrom
Vialathor:parse-ubuntu-suffix
Jun 27, 2025
Merged

fix: Parse ubuntu suffix#1982
hogo6002 merged 5 commits intogoogle:mainfrom
Vialathor:parse-ubuntu-suffix

Conversation

@Vialathor
Copy link
Contributor

Removed ":Pro" ":LTS" from appearing in string comparison method for getNextFixVersion(...) for Ubuntu environments.

@cuixq cuixq requested a review from hogo6002 June 19, 2025 23:06
@hogo6002
Copy link
Contributor

can you update branch and then run the test case locally to update the snapshots?

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.37%. Comparing base (c405556) to head (dccc8f2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1982      +/-   ##
==========================================
+ Coverage   65.34%   65.37%   +0.02%     
==========================================
  Files         176      176              
  Lines       17174    17180       +6     
==========================================
+ Hits        11222    11231       +9     
+ Misses       5188     5187       -1     
+ Partials      764      762       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hogo6002 hogo6002 merged commit bd65986 into google:main Jun 27, 2025
16 checks passed
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.

5 participants