Skip to content

Commit

Permalink
Fix CI, and add lint test
Browse files Browse the repository at this point in the history
  • Loading branch information
gsabran authored Dec 18, 2024
2 parents c55aa03 + 58b9ce4 commit cb57bcb
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 4 deletions.
8 changes: 8 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
## Summary
<!--- Provide a general summary of your changes in the Title above -->

## Details
<!--- Describe your changes in detail -->

## Checklist before merging
- [ ] If this is a new feature or a bug fix, I have added tests.
27 changes: 23 additions & 4 deletions .github/workflows/swift.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,32 @@ on:
branches: [ "main" ]

jobs:
build:
build_and_test:

runs-on: macos-latest
# TODO: once https://github.com/swift-actions/setup-swift/pull/684 is merged, revert to:

# runs-on: macos-latest
# steps:
# - uses: swift-actions/setup-swift@v2
# with:
# swift-version: "6.0.1"

runs-on: macos-15

steps:
- name: Get swift version
run: swift --version
- uses: actions/checkout@v4
- name: Build
run: swift build -v
run: swift build -q
- name: Run tests
run: swift test -v
run: swift test -q

lint:
runs-on: macos-15
steps:
- uses: actions/checkout@v4
- name: Run linter
run: swift package --allow-writing-to-package-directory format
- name: Verify that `swift package --allow-writing-to-package-directory format` did not change outputs (if it did, please re-run it and re-commit!)
run: git diff --exit-code

0 comments on commit cb57bcb

Please sign in to comment.