Skip to content

Restore and deprecate TypeOptionDiscovered#348

Merged
yevhenii-nadtochii merged 4 commits intomasterfrom
restore-type-option-discovered
Apr 15, 2025
Merged

Restore and deprecate TypeOptionDiscovered#348
yevhenii-nadtochii merged 4 commits intomasterfrom
restore-type-option-discovered

Conversation

@yevhenii-nadtochii
Copy link
Contributor

@yevhenii-nadtochii yevhenii-nadtochii commented Apr 15, 2025

This PR aims to fix the binary compatibility with the older ProtoData from mc-java. In validation, it brings the descriptors for the removed type, but without the Java type itself.

TypeOptionDiscovered is restored as deprecated.

@yevhenii-nadtochii yevhenii-nadtochii self-assigned this Apr 15, 2025
@yevhenii-nadtochii
Copy link
Contributor Author

@armiol @alexander-yevsyukov PTAL

@yevhenii-nadtochii yevhenii-nadtochii marked this pull request as ready for review April 15, 2025 16:25
@codecov
Copy link

codecov bot commented Apr 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.82%. Comparing base (6f966a5) to head (abf7f14).
Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #348   +/-   ##
=========================================
  Coverage     74.82%   74.82%           
  Complexity      615      615           
=========================================
  Files           196      196           
  Lines          4254     4254           
  Branches        396      396           
=========================================
  Hits           3183     3183           
  Misses          938      938           
  Partials        133      133           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yevhenii-nadtochii yevhenii-nadtochii merged commit 2ef601e into master Apr 15, 2025
8 checks passed
@yevhenii-nadtochii yevhenii-nadtochii deleted the restore-type-option-discovered branch April 15, 2025 16:43
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