Thanks to visit codestin.com
Credit goes to github.com

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

Revert "ci: Java 17 profile to run tests with old Mockito (#1343)"#1349

Merged
tom-andersen merged 1 commit into
mainfrom
revert-1343-i1342
Jul 5, 2023
Merged

Revert "ci: Java 17 profile to run tests with old Mockito (#1343)"#1349
tom-andersen merged 1 commit into
mainfrom
revert-1343-i1342

Conversation

@tom-andersen

@tom-andersen tom-andersen commented Jul 4, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

A more permanent fix was done in #1345

So we can revert #1343

@tom-andersen tom-andersen requested review from a team July 4, 2023 14:03
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: firestore Issues related to the googleapis/java-firestore API. labels Jul 4, 2023
@tom-andersen tom-andersen added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 4, 2023
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 4, 2023
@suztomo

suztomo commented Jul 5, 2023

Copy link
Copy Markdown
Member

I see javadoc failing.

Error:  /home/runner/work/java-firestore/java-firestore/google-cloud-firestore/src/main/java/com/google/cloud/firestore/FirestoreBundle.java:136: error: unknown tag: returns
Error:       * @returns This instance.
Error:         ^

Would you fix them? (If you do, focus on "error:" not "warning: ")

@tom-andersen

tom-andersen commented Jul 5, 2023

Copy link
Copy Markdown
Contributor Author

I see javadoc failing.

Error:  /home/runner/work/java-firestore/java-firestore/google-cloud-firestore/src/main/java/com/google/cloud/firestore/FirestoreBundle.java:136: error: unknown tag: returns
Error:       * @returns This instance.
Error:         ^

Would you fix them? (If you do, focus on "error:" not "warning: ")

Is this a new check?

I would like to fix this in a separate PR in order to keep this PR a clean revert of previous PR. I think introducing more changes here will muddy the waters.

suztomo
suztomo approved these changes Jul 5, 2023
View reviewed changes

@suztomo suztomo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sounds good.

@tom-andersen tom-andersen merged commit 7ff8e0d into main Jul 5, 2023
@tom-andersen tom-andersen deleted the revert-1343-i1342 branch July 5, 2023 15:43
@suztomo

suztomo commented Jul 5, 2023

Copy link
Copy Markdown
Member

@tom-andersen The release job may fail if javadoc fails. (The release job needs to generate javadoc for Maven Central)

cherylEnkidu pushed a commit that referenced this pull request Dec 11, 2023
@tom-andersen @cherylEnkidu
Revert "ci: Java 17 profile to run tests with old Mockito (#1343)" (#…
ec0193e 
…1349)

This reverts commit 035adde.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@MarkDuckworth MarkDuckworth Awaiting requested review from MarkDuckworth
1 more reviewer
@suztomo suztomo suztomo approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@suztomo suztomo

@MarkDuckworth MarkDuckworth

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tom-andersen @suztomo @MarkDuckworth @yoshi-kokoro