feat: add org.json:json to third-party-dependencies pom by chingor13 · Pull Request #4047 · googleapis/sdk-platform-java · GitHub
Skip to content
This repository was archived by the owner on May 14, 2026. It is now read-only.

feat: add org.json:json to third-party-dependencies pom#4047

Merged
blakeli0 merged 5 commits into
mainfrom
add-json
Jan 9, 2026
Merged

feat: add org.json:json to third-party-dependencies pom#4047
blakeli0 merged 5 commits into
mainfrom
add-json

Conversation

@chingor13

Copy link
Copy Markdown
Contributor

Fixes #4046

@chingor13 chingor13 requested a review from blakeli0 January 8, 2026 23:42
@product-auto-label product-auto-label Bot added the size: xs Pull request size is extra small. label Jan 8, 2026
@gemini-code-assist

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds the org.json:json dependency to the shared third-party dependencies POM. My main concern is the version specified for this new dependency, 20251224, which does not exist in Maven Central and will cause build failures. I've suggested updating it to the latest stable version. I've also included some suggestions to improve maintainability by keeping properties and dependencies sorted alphabetically in the POM file.

Comment thread java-shared-dependencies/third-party-dependencies/pom.xml Outdated
Comment thread java-shared-dependencies/third-party-dependencies/pom.xml Outdated
Comment thread java-shared-dependencies/third-party-dependencies/pom.xml Outdated
Comment thread java-shared-dependencies/third-party-dependencies/pom.xml Outdated

@blakeli0 blakeli0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mind adding org.json:json to the dependencies.txt and use the latest version 20251224 there? We use this file to test if our client libraries are compatible with the latest dependencies.

Comment thread dependencies.txt Outdated
@sonarqubecloud

sonarqubecloud Bot commented Jan 9, 2026

Copy link
Copy Markdown

@sonarqubecloud

sonarqubecloud Bot commented Jan 9, 2026

Copy link
Copy Markdown

@blakeli0 blakeli0 merged commit ffa432e into main Jan 9, 2026
62 of 65 checks passed
@blakeli0 blakeli0 deleted the add-json branch January 9, 2026 20:57
diegomarquezp pushed a commit that referenced this pull request Jan 13, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>2.65.0</summary>

##
[2.65.0](v2.64.2...v2.65.0)
(2026-01-12)


### Features

* add org.json:json to third-party-dependencies pom
([#4047](#4047))
([ffa432e](ffa432e)),
closes
[#4046](#4046)
* remove dependency management of graal-sdk
([#4033](#4033))
([ad05c34](ad05c34))


### Bug Fixes

* add api_version breadcrumb to client docs
([#4018](#4018))
([a2b2179](a2b2179))
* Create a single S2AChannelCredentials per application
([#3989](#3989))
([3758b43](3758b43))
* provide API to share the same background executor for channel po…
([#4030](#4030))
([178182c](178182c))


### Dependencies

* update dependencies.txt for grpc-gcp to 1.9.0
([#4025](#4025))
([b68791d](b68791d))
* update google api dependencies
([#3917](#3917))
([480cf13](480cf13))
* update google.http-client.version to 2.0.3
([#4054](#4054))
([b9a8c89](b9a8c89))


### Documentation

* Fix retry guide link in javadocs
([#4029](#4029))
([b43f77c](b43f77c))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
diegomarquezp pushed a commit that referenced this pull request Jan 21, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>2.65.0</summary>

##
[2.65.0](v2.64.2...v2.65.0)
(2026-01-12)


### Features

* add org.json:json to third-party-dependencies pom
([#4047](#4047))
([ffa432e](ffa432e)),
closes
[#4046](#4046)
* remove dependency management of graal-sdk
([#4033](#4033))
([ad05c34](ad05c34))


### Bug Fixes

* add api_version breadcrumb to client docs
([#4018](#4018))
([a2b2179](a2b2179))
* Create a single S2AChannelCredentials per application
([#3989](#3989))
([3758b43](3758b43))
* provide API to share the same background executor for channel po…
([#4030](#4030))
([178182c](178182c))


### Dependencies

* update dependencies.txt for grpc-gcp to 1.9.0
([#4025](#4025))
([b68791d](b68791d))
* update google api dependencies
([#3917](#3917))
([480cf13](480cf13))
* update google.http-client.version to 2.0.3
([#4054](#4054))
([b9a8c89](b9a8c89))


### Documentation

* Fix retry guide link in javadocs
([#4029](#4029))
([b43f77c](b43f77c))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
lqiu96 pushed a commit that referenced this pull request Feb 12, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>2.65.0</summary>

##
[2.65.0](v2.64.2...v2.65.0)
(2026-01-12)


### Features

* add org.json:json to third-party-dependencies pom
([#4047](#4047))
([ffa432e](ffa432e)),
closes
[#4046](#4046)
* remove dependency management of graal-sdk
([#4033](#4033))
([ad05c34](ad05c34))


### Bug Fixes

* add api_version breadcrumb to client docs
([#4018](#4018))
([a2b2179](a2b2179))
* Create a single S2AChannelCredentials per application
([#3989](#3989))
([3758b43](3758b43))
* provide API to share the same background executor for channel po…
([#4030](#4030))
([178182c](178182c))


### Dependencies

* update dependencies.txt for grpc-gcp to 1.9.0
([#4025](#4025))
([b68791d](b68791d))
* update google api dependencies
([#3917](#3917))
([480cf13](480cf13))
* update google.http-client.version to 2.0.3
([#4054](#4054))
([b9a8c89](b9a8c89))


### Documentation

* Fix retry guide link in javadocs
([#4029](#4029))
([b43f77c](b43f77c))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
chingor13 pushed a commit to googleapis/google-cloud-java that referenced this pull request Mar 24, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>2.65.0</summary>

##
[2.65.0](googleapis/sdk-platform-java@v2.64.2...v2.65.0)
(2026-01-12)


### Features

* add org.json:json to third-party-dependencies pom
([#4047](googleapis/sdk-platform-java#4047))
([5898abd](googleapis/sdk-platform-java@5898abd)),
closes
[#4046](googleapis/sdk-platform-java#4046)
* remove dependency management of graal-sdk
([#4033](googleapis/sdk-platform-java#4033))
([09ec2a3](googleapis/sdk-platform-java@09ec2a3))


### Bug Fixes

* add api_version breadcrumb to client docs
([#4018](googleapis/sdk-platform-java#4018))
([0573232](googleapis/sdk-platform-java@0573232))
* Create a single S2AChannelCredentials per application
([#3989](googleapis/sdk-platform-java#3989))
([6a24164](googleapis/sdk-platform-java@6a24164))
* provide API to share the same background executor for channel po…
([#4030](googleapis/sdk-platform-java#4030))
([051f76a](googleapis/sdk-platform-java@051f76a))


### Dependencies

* update dependencies.txt for grpc-gcp to 1.9.0
([#4025](googleapis/sdk-platform-java#4025))
([2376f6c](googleapis/sdk-platform-java@2376f6c))
* update google api dependencies
([#3917](googleapis/sdk-platform-java#3917))
([bf94a0a](googleapis/sdk-platform-java@bf94a0a))
* update google.http-client.version to 2.0.3
([#4054](googleapis/sdk-platform-java#4054))
([343a8ef](googleapis/sdk-platform-java@343a8ef))


### Documentation

* Fix retry guide link in javadocs
([#4029](googleapis/sdk-platform-java#4029))
([4bb3ccc](googleapis/sdk-platform-java@4bb3ccc))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add org.json:json to managed third-party dependencies

2 participants