Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix broken gradle repo URL #6917

Closed
wants to merge 1 commit into from

Conversation

steventrouble
Copy link

/gradle/ doesn't work anymore, but /artifactory/ works.

The PR is in accordance with the Developer's Certificate of Origin.


Tested: ./gradlew build && ./gradlew test

Before:

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':node-driver:compileKotlin'.
> Could not resolve all files for configuration ':node-driver:compileClasspath'.
   > Could not find org.gradle:gradle-tooling-api:5.6.4.

After:

net.corda.core.crypto.CryptoUtilsTest > test default SecureRandom uses platformSecureRandom FAILED
210 tests completed, 1 failed, 3 skipped

/gradle/ doesn't work anymore, but /artifactory/ works.
@nargas-ritu
Copy link
Contributor

Internal ticket CORDA-4187

@marcellofarina75
Copy link

Hello steventrouble,
thank you for your contribution to Corda!

Our engineers reviewed your Pull Request and established that, while your proposal would definitely improve Corda, the work needed to merge it (in terms of unit, integration and regression tests, documentation updates, security assessments, and ongoing maintenance) would prevent us from shipping features and fixes that are prioritised higher in our roadmap (which is driven directly from inputs from across the user base).

So this means we regretfully need to let you know that your PR will be declined.

Please feel free to contact devrel@r3.com if you need any further information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants