Skip to content

Update dependencies, fix build#653

Merged
yahavi merged 2 commits intojfrog:masterfrom
elevenfive:update-dependencies-fix-build
Jul 3, 2022
Merged

Update dependencies, fix build#653
yahavi merged 2 commits intojfrog:masterfrom
elevenfive:update-dependencies-fix-build

Conversation

@elevenfive
Copy link
Copy Markdown
Contributor

@elevenfive elevenfive commented Jun 13, 2022 •
edited
Loading

  • All tests passed. If this feature is not already covered by the tests, I added new tests.

Have run all the tests that don't require a docker instance running. If someone else has one handy and can test this, it would be appreciated!

Fixes #652

@elevenfive elevenfive marked this pull request as ready for review June 13, 2022 22:52
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 13, 2022 •
edited
Loading

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@elevenfive
Copy link
Copy Markdown
Contributor Author

elevenfive commented Jun 13, 2022

I have read the CLA Document and I hereby sign the CLA

@elevenfive
Copy link
Copy Markdown
Contributor Author

elevenfive commented Jun 13, 2022

recheck

@elevenfive elevenfive changed the title Update dependencies Update dependencies, fix build Jun 13, 2022
@eyalbe4 eyalbe4 self-requested a review June 13, 2022 23:05
@alivesubstance
Copy link
Copy Markdown

alivesubstance commented Jun 23, 2022

Guys, any updates on this PR?

yahavi
yahavi approved these changes Jul 3, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@yahavi yahavi left a comment

Choose a reason for hiding this comment

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

Thanks, @elevenfive!

@yahavi yahavi merged commit 9548db6 into jfrog:master Jul 3, 2022
yahavi added a commit to yahavi/build-info that referenced this pull request Jul 3, 2022
@yahavi
Revert "Update dependencies, fix build (jfrog#653)"
2bf7e1f 
This reverts commit 9548db6.
yahavi added a commit that referenced this pull request Jul 3, 2022
@yahavi
Revert "Update dependencies, fix build (#653)"
bf41c53 
This reverts commit 9548db6.
@yahavi
Copy link
Copy Markdown
Member

yahavi commented Jul 3, 2022 •
edited
Loading

@elevenfive,
After merging this PR, I noticed that the publish tasks failed. As a result, I reverted this change.
We'd appreciate it if you could create a new PR after verifying that the publishToMavenLocal task passes.

By the way - what is the original reason that the build failed for you? Did you compile with JDK 8?
Thanks.

@elevenfive
Copy link
Copy Markdown
Contributor Author

elevenfive commented Jul 4, 2022 •
edited
Loading

Hi I was using JDK 11; when attempting ./gradlew build on master, I was getting > Task :build-info-api:compileJava FAILED

When switching to JDK 8 I am able to build current master, thank you!

Upgrading to Gradle 7 was the reason for the publish failure on this branch, sorry for not checking that task!

I will push with Gradle 6 to #659. Gradle 7 migration will be more involved and beyond my current expertise with Gradle plugins.

@elevenfive elevenfive mentioned this pull request Jul 4, 2022
Merged
1 task
@elevenfive elevenfive deleted the update-dependencies-fix-build branch July 4, 2022 20:32
@kilinochi
Copy link
Copy Markdown

kilinochi commented Jul 6, 2022 •
edited
Loading

hello! when you are going to collect new realize ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yahavi yahavi yahavi approved these changes

@eyalbe4 eyalbe4 Awaiting requested review from eyalbe4

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Project has out of date dependencies and './gradlew clean build test' fails

4 participants

@elevenfive @alivesubstance @yahavi @kilinochi