Skip to content

Mark as many classes as final as possible#557

Merged
Ostrzyciel merged 1 commit intomainfrom
piotr/make-things-final
Sep 30, 2025
Merged

Mark as many classes as final as possible#557
Ostrzyciel merged 1 commit intomainfrom
piotr/make-things-final

Conversation

@Ostrzyciel
Copy link
Member

This can increase performance in some cases... but in general, it helps with communicating to people that no, these things are not intended to be extended.

@Ostrzyciel Ostrzyciel enabled auto-merge (squash) September 30, 2025 12:21
@codecov
Copy link

codecov bot commented Sep 30, 2025

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...u/neverblink/protoc/java/runtime/ProtobufUtil.java 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ Complexity Δ
...neverblink/jelly/core/patch/JellyPatchOptions.java 96.87% <ø> (ø) 9.00 <0.00> (ø)
...nk/jelly/core/patch/internal/PatchDecoderImpl.java 93.50% <ø> (ø) 41.00 <0.00> (ø)
...nk/jelly/core/patch/internal/PatchEncoderImpl.java 100.00% <ø> (ø) 22.00 <0.00> (ø)
...in/java/eu/neverblink/jelly/core/JellyOptions.java 100.00% <ø> (ø) 10.00 <0.00> (ø)
...verblink/jelly/core/internal/BaseJellyOptions.java 97.14% <ø> (ø) 10.00 <0.00> (ø)
.../neverblink/jelly/core/internal/DecoderLookup.java 80.00% <ø> (ø) 3.00 <0.00> (ø)
...verblink/jelly/core/internal/ProtoEncoderImpl.java 100.00% <ø> (ø) 13.00 <0.00> (ø)
...blink/jelly/core/internal/ProtoTranscoderImpl.java 97.19% <ø> (ø) 59.00 <0.00> (ø)
...rblink/jelly/convert/jena/patch/JellyPatchOps.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...ert/jena/riot/JellyStreamWriterAutodetectType.java 97.91% <ø> (ø) 0.00 <0.00> (ø)
... and 13 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Ostrzyciel Ostrzyciel merged commit e4b71fa into main Sep 30, 2025
12 checks passed
@Ostrzyciel Ostrzyciel deleted the piotr/make-things-final branch September 30, 2025 12:38
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