Skip to content

Commit

Permalink
CI Updates (vapor#96)
Browse files Browse the repository at this point in the history
* Update exports
* Remove CodeQL as it's part of our standard CI
* Some additional CI cleanup
  • Loading branch information
0xTim authored Oct 9, 2023
1 parent fd88cee commit d3f099d
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/api-docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ jobs:
with:
package_name: jwt-kit
modules: JWTKit
pathsToInvalidate: /jwt-kit
pathsToInvalidate: /jwt-kit/*
11 changes: 4 additions & 7 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@ jobs:
linux-integration:
if: ${{ !(github.event.pull_request.draft || false) }}
runs-on: ubuntu-latest
container: swift:5.8-jammy
container: swift:jammy
steps:
- name: Check out JWTKit
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
path: jwt-kit
- name: Check out JWT provider
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: vapor/jwt
path: jwt
Expand All @@ -27,7 +27,4 @@ jobs:
run: swift test --package-path jwt --sanitize=thread

unit-tests:
uses: vapor/ci/.github/workflows/run-unit-tests.yml@reusable-workflows
with:
with_coverage: true
with_tsan: true
uses: vapor/ci/.github/workflows/run-unit-tests.yml@main
8 changes: 7 additions & 1 deletion Sources/JWTKit/Utilities/Exports.swift
Original file line number Diff line number Diff line change
@@ -1,4 +1,10 @@
#if !BUILDING_DOCC
#if swift(>=5.8)

@_documentation(visibility: internal) @_exported import struct Foundation.Date
@_documentation(visibility: internal) @_exported import protocol Foundation.DataProtocol
@_documentation(visibility: internal) @_exported import protocol Foundation.ContiguousBytes

#else

@_exported import struct Foundation.Date
@_exported import protocol Foundation.DataProtocol
Expand Down

0 comments on commit d3f099d

Please sign in to comment.