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

[build] migrate to Gradle #177

Merged
merged 3 commits into from
May 26, 2022
Merged

Conversation

dariuszkuc
Copy link
Member

Resolves #131

@dariuszkuc dariuszkuc added the type: build Build/tooling related changes label May 25, 2022
Explicitly checking for SNAPSHOT version as extra properties are not available at java task configuration time.
Copy link
Contributor

@martinbonnin martinbonnin left a comment

Choose a reason for hiding this comment

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

I'm off for a long week end so just made a very quick pass at this but I don't want to block it, LGTM! Super happy this is happening 🐘 🚀

@dariuszkuc dariuszkuc merged commit 0fedd89 into apollographql:master May 26, 2022
@dariuszkuc dariuszkuc deleted the build branch May 26, 2022 19:17
version = 2.0.2-SNAPSHOT

# See https://github.com/gradle/gradle/issues/11308
systemProp.org.gradle.internal.publish.checksums.insecure=true
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you can skip this. Sonatype now handles this correctly

theJC pushed a commit to theJC/federation-jvm that referenced this pull request Jul 15, 2022
* [build] migrate to Gradle
Resolves apollographql#131

* generate sources/javadocs only for releases

* conditionally build sources and javadocs
Explicitly checking for SNAPSHOT version as extra properties are not available at java task configuration time.

Co-authored-by: Dariusz Kuc <dkuc@apollographql.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: build Build/tooling related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move build system to Gradle?
2 participants