-
Notifications
You must be signed in to change notification settings - Fork 1.7k
AVRO-3731: [gradle-avro-plugin] Integrate software donation of gradle-avro-plugin #3305
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
base: main
Are you sure you want to change the base?
Changes from all commits
cd9a7a0
b41d607
499a1b1
da0e6a0
dfc706c
eb2bc46
9d94688
c2b410b
848097c
1265a53
0545f04
97a499a
b3122a7
b6963e3
2a15e3f
df17d97
f610c4c
9a7e883
d7f11e6
156e993
90ab778
6b3fcbc
dd742a8
5111231
b2b364c
851e341
8d5a52f
4a16f45
64ed2cc
4510296
4ef9725
76e3525
a040edd
3dcafa3
65d2f4d
ad2ad3f
0499977
139e4c1
5b4290c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,42 @@ | ||
| # Licensed to the Apache Software Foundation (ASF) under one or more | ||
| # contributor license agreements. See the NOTICE file distributed with | ||
| # this work for additional information regarding copyright ownership. | ||
| # The ASF licenses this file to You under the Apache License, Version 2.0 | ||
| # (the "License"); you may not use this file except in compliance with | ||
| # the License. You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # For most projects, this workflow file will not need changing; you simply need | ||
| # to commit it to your repository. | ||
|
|
||
| # You may wish to alter this file to override the set of languages analyzed, | ||
| # or to provide custom queries or build logic. | ||
|
|
||
| name: Gradle Compatibility Tests | ||
| on: [ pull_request ] | ||
| jobs: | ||
| test: | ||
| name: "Compatibility: gradle ${{ matrix.gradle }}, java ${{ matrix.java }}, avro ${{matrix.avro}}" | ||
| runs-on: "ubuntu-latest" | ||
| strategy: | ||
| matrix: | ||
| avro: [ "1.12.0" ] | ||
| gradle: [ "8.14" ] | ||
| java: [ "17" ] | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-java@v4 | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Those are outdated but Dependabot will upgrade them once this PR is merged |
||
| with: | ||
| distribution: "zulu" | ||
| java-version: ${{ matrix.java }} | ||
| - uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| build-root-directory: lang/java/gradle-plugin | ||
| gradle-executable: lang/java/gradle-plugin/gradlew | ||
| arguments: testCompatibility -PavroVersion=${{ matrix.avro }} -PgradleVersion=${{ matrix.gradle }} | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,43 @@ | ||
| # Licensed to the Apache Software Foundation (ASF) under one or more | ||
| # contributor license agreements. See the NOTICE file distributed with | ||
| # this work for additional information regarding copyright ownership. | ||
| # The ASF licenses this file to You under the Apache License, Version 2.0 | ||
| # (the "License"); you may not use this file except in compliance with | ||
| # the License. You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # For most projects, this workflow file will not need changing; you simply need | ||
| # to commit it to your repository. | ||
|
|
||
| # You may wish to alter this file to override the set of languages analyzed, | ||
| # or to provide custom queries or build logic. | ||
|
|
||
| name: Gradle plugin build | ||
| on: [ pull_request ] | ||
| jobs: | ||
| build: | ||
| name: "Build Gradle plugin" | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-java@v4 | ||
| with: | ||
| distribution: 'temurin' | ||
| java-version: 17 | ||
| - name: Setup Gradle | ||
| uses: gradle/actions/setup-gradle@v3 | ||
| - name: debug pwd | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. debug leftovers |
||
| run: | | ||
| echo "pwd:" | ||
| pwd | ||
| echo "ls" | ||
| ls -la | ||
| - name: Execute Gradle build | ||
| working-directory: lang/java/gradle-plugin | ||
| run: ./gradlew build | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| # Licensed to the Apache Software Foundation (ASF) under one or more | ||
| # contributor license agreements. See the NOTICE file distributed with | ||
| # this work for additional information regarding copyright ownership. | ||
| # The ASF licenses this file to You under the Apache License, Version 2.0 | ||
| # (the "License"); you may not use this file except in compliance with | ||
| # the License. You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # For most projects, this workflow file will not need changing; you simply need | ||
| # to commit it to your repository. | ||
|
|
||
| # You may wish to alter this file to override the set of languages analyzed, | ||
| # or to provide custom queries or build logic. | ||
|
|
||
| # See https://github.com/marketplace/actions/gradle-wrapper-validation | ||
| name: "Validate Gradle Wrapper" | ||
| on: [ pull_request ] | ||
|
|
||
| jobs: | ||
| validation: | ||
| name: "Validation" | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: gradle/actions/wrapper-validation@v3 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,64 @@ | ||
| # Licensed to the Apache Software Foundation (ASF) under one or more | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's rename this file to |
||
| # contributor license agreements. See the NOTICE file distributed with | ||
| # this work for additional information regarding copyright ownership. | ||
| # The ASF licenses this file to You under the Apache License, Version 2.0 | ||
| # (the "License"); you may not use this file except in compliance with | ||
| # the License. You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # For most projects, this workflow file will not need changing; you simply need | ||
| # to commit it to your repository. | ||
|
|
||
| # You may wish to alter this file to override the set of languages analyzed, | ||
| # or to provide custom queries or build logic. | ||
|
|
||
| # See https://docs.gradle.org/current/userguide/compatibility.html | ||
| name: Gradle Java Compatibility Tests | ||
| on: [ pull_request ] | ||
| jobs: | ||
| java17: | ||
| name: "Compatibility: java ${{ matrix.java }}/gradle ${{ matrix.gradle }}" | ||
| runs-on: "ubuntu-latest" | ||
| strategy: | ||
| matrix: | ||
| avro: [ "1.12.0" ] | ||
| gradle: [ "8.14" ] | ||
| java: [ "17" ] | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-java@v4 | ||
| with: | ||
| distribution: "zulu" | ||
| java-version: ${{ matrix.java }} | ||
| - uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| build-root-directory: lang/java/gradle-plugin | ||
| gradle-executable: lang/java/gradle-plugin/gradlew | ||
| arguments: testCompatibility -PavroVersion=${{ matrix.avro }} -PgradleVersion=${{ matrix.gradle }} | ||
| java-ea: | ||
| name: "Compatibility: java ${{ matrix.java }}/gradle ${{ matrix.gradle }}" | ||
| runs-on: "ubuntu-latest" | ||
| strategy: | ||
| matrix: | ||
| avro: [ "1.12.0" ] | ||
| gradle: [ "8.14" ] | ||
| java: [ "21" ] | ||
| fail-fast: false | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-java@v4 | ||
| with: | ||
| distribution: "zulu" | ||
| java-version: ${{ matrix.java }} | ||
| - uses: gradle/gradle-build-action@v2 | ||
| continue-on-error: true | ||
| with: | ||
| build-root-directory: lang/java/gradle-plugin | ||
| gradle-executable: lang/java/gradle-plugin/gradlew | ||
| arguments: testCompatibility -PavroVersion=${{ matrix.avro }} -PgradleVersion=${{ matrix.gradle }} | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,41 @@ | ||
| # Licensed to the Apache Software Foundation (ASF) under one or more | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's rename this file to |
||
| # contributor license agreements. See the NOTICE file distributed with | ||
| # this work for additional information regarding copyright ownership. | ||
| # The ASF licenses this file to You under the Apache License, Version 2.0 | ||
| # (the "License"); you may not use this file except in compliance with | ||
| # the License. You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # For most projects, this workflow file will not need changing; you simply need | ||
| # to commit it to your repository. | ||
|
|
||
| # You may wish to alter this file to override the set of languages analyzed, | ||
| # or to provide custom queries or build logic. | ||
|
|
||
| name: OS Compatibility | ||
| on: [ pull_request ] | ||
| jobs: | ||
| build: | ||
| name: "Compatibility: ${{ matrix.os }}" | ||
| runs-on: ${{ matrix.os }} | ||
| strategy: | ||
| matrix: | ||
| java: [ "17" ] | ||
| os: [ ubuntu-latest, windows-latest, macOS-latest ] # All supported OS | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-java@v4 | ||
| with: | ||
| distribution: zulu | ||
| java-version: ${{ matrix.java }} | ||
| - uses: gradle/gradle-build-action@v2 | ||
| with: | ||
| build-root-directory: lang/java/gradle-plugin | ||
| gradle-executable: lang/java/gradle-plugin/gradlew | ||
| arguments: test | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| # EditorConfig is awesome: http://EditorConfig.org | ||
|
|
||
| root = true | ||
|
|
||
| [*] | ||
| end_of_line = lf | ||
| insert_final_newline = true | ||
| charset = utf-8 | ||
| indent_style = space | ||
| indent_size = 4 | ||
|
|
||
| [*.yml] | ||
| indent_size = 2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this automatically use the latest patch version of 8.14 ? I.e. 8.14.3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be a good idea to use Gradle 9.x ?