Skip to content
This repository was archived by the owner on Feb 6, 2026. It is now read-only.

fix: Reintroduce JSpecify annotations.#1709

Closed
cpovirk wants to merge 1 commit intogoogleapis:mainfrom
cpovirk:jsagain
Closed

fix: Reintroduce JSpecify annotations.#1709
cpovirk wants to merge 1 commit intogoogleapis:mainfrom
cpovirk:jsagain

Conversation

@cpovirk
Copy link

@cpovirk cpovirk commented Oct 15, 2024

JSpecify 1.0.0 has been released, so I'm hoping that
#1362 is no longer a
concern. (If there are remaining concerns, please do let me know.)

This reverts commit 8138f46 (aka
#1364) with a few
changes:

(JSpecify annotations were originally introduced by
#1010.)

@cpovirk cpovirk requested review from a team and cindy-peng October 15, 2024 19:16
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: logging Issues related to the googleapis/java-logging API. labels Oct 15, 2024
JSpecify 1.0.0 has been released, so I'm hoping that
googleapis#1362 is no longer a
concern. (If there are remaining concerns, please do let me know.)

This reverts commit 8138f46 (aka
googleapis#1364) with a few
changes:

- This commit uses JSpecify 1.0.0 instead of 0.2.0.
- As a result, it uses `org.jspecify.annotations` instead of
  `org.jspecify.nullness`.
- It no longer touches `Structs`, since that was deleted in
  googleapis#1501.

(JSpecify annotations were originally introduced by
googleapis#1010.)
@cpovirk cpovirk changed the title Reintroduce JSpecify annotations. fix: Reintroduce JSpecify annotations. Oct 15, 2024
@chingor13 chingor13 closed this Feb 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: logging Issues related to the googleapis/java-logging API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants