KEMBAR78
fix: declare gson as runtime rather than test by suztomo · Pull Request #1004 · googleapis/java-firestore · GitHub
Skip to content

Conversation

@suztomo
Copy link
Member

@suztomo suztomo commented Aug 8, 2022

Fixing similar issues as googleapis/java-pubsub#1239.

In this case, declaring gson as test causes the flatten-maven-plugin to exclude gson from other dependency declarations.

@suztomo suztomo requested review from a team as code owners August 8, 2022 18:14
@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/java-firestore API. label Aug 8, 2022
@suztomo suztomo requested a review from blakeli0 August 8, 2022 18:14
@alicejli alicejli merged commit e37e997 into main Aug 19, 2022
@alicejli alicejli deleted the suztomo-patch-1 branch August 19, 2022 17:47
gcf-merge-on-green bot pushed a commit that referenced this pull request Aug 19, 2022
🤖 I have created a release *beep* *boop*
---


## [3.4.1](v3.4.0...v3.4.1) (2022-08-19)


### Bug Fixes

* declare gson as runtime rather than test ([#1004](#1004)) ([e37e997](e37e997))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
cherylEnkidu pushed a commit that referenced this pull request Dec 11, 2023
cherylEnkidu pushed a commit that referenced this pull request Dec 11, 2023
🤖 I have created a release *beep* *boop*
---


## [3.4.1](v3.4.0...v3.4.1) (2022-08-19)


### Bug Fixes

* declare gson as runtime rather than test ([#1004](#1004)) ([e37e997](e37e997))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the googleapis/java-firestore API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants