Skip to content

Commit

Permalink
refactor(ci): Adjust build process execution in the Snapshot Workflow (
Browse files Browse the repository at this point in the history
…#300)

- Exclude 'test' step from 'build' job
- Execute build for both push and pull request scenarios
- Additionally, include 'Verify CLI' step for pull requests
  • Loading branch information
puneetbehl authored Mar 15, 2024
1 parent caade8e commit dc852a9
Showing 1 changed file with 1 addition and 12 deletions.
13 changes: 1 addition & 12 deletions .github/workflows/snapshot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,18 +20,7 @@ jobs:
with:
distribution: 'adopt'
java-version: ${{ matrix.java }}
- name: Run Tests
if: github.event_name == 'pull_request'
id: tests
uses: gradle/gradle-build-action@v2
env:
GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GRADLE_ENTERPRISE_ACCESS_KEY }}
GRADLE_ENTERPRISE_BUILD_CACHE_NODE_USER: ${{ secrets.GRADLE_ENTERPRISE_BUILD_CACHE_NODE_USER }}
GRADLE_ENTERPRISE_BUILD_CACHE_NODE_KEY: ${{ secrets.GRADLE_ENTERPRISE_BUILD_CACHE_NODE_KEY }}
with:
arguments: check
- name: Run Build
if: github.event_name == 'push'
id: build
uses: gradle/gradle-build-action@v2
env:
Expand All @@ -41,7 +30,7 @@ jobs:
with:
arguments: build
- name: Verify CLI
if: steps.build.outcome == 'success' && github.event_name == 'push'
if: success()
run: |
cp grails-cli/build/distributions/grails-cli-*.zip cli.zip
unzip cli -d tmp
Expand Down

0 comments on commit dc852a9

Please sign in to comment.