Skip to content

Replace dead Travis CI with GitHub Actions - #24

Merged
jbuckner merged 4 commits into
mainfrom
claude/serene-neumann-f9481b
Jun 19, 2026
Merged

jbuckner merged 4 commits into
mainfrom
claude/serene-neumann-f9481b

Conversation

@jbuckner

Copy link
Copy Markdown
Owner

Summary

  • Travis CI stopped building this repo when its free OSS tier shut down (last real CI run was around PR Add support for async/await #17, Dec 2021), so recent PRs merged with no checks. This replaces the inert .travis.yml with a GitHub Actions workflow.
  • Build and Test job: swift test --enable-code-coverage on macos-latest, with lcov export uploaded to Codecov (tokenless; add a CODECOV_TOKEN secret if uploads get rate-limited).
  • SwiftLint job: pinned ghcr.io/realm/swiftlint:0.63.3 container on ubuntu.
  • Fixes the package so it actually builds again: bumps the macOS platform minimum from 10.13 to 10.15 — the async/await public API already requires 10.15, and the os_log(_:log:_:_:) overload requires 10.14, which made swift test fail to compile on current toolchains.
  • Lint config: exclude .build/ (dependency checkouts were being linted) and exempt reviewer_itemname from identifier_name since it mirrors the Internet Archive API field.
  • README: swap the Travis badge for a GitHub Actions badge and point the codecov badge at main instead of master.

Test plan

  • swift test --enable-code-coverage passes locally (63 tests, 0 failures)
  • lcov export command verified locally
  • SwiftLint 0.63.3 passes locally (0 error-level violations)
  • CI workflow runs green on this PR

🤖 Generated with Claude Code

jbuckner and others added 4 commits June 11, 2026 23:48
Travis CI stopped building this repo when its free OSS tier shut down,
leaving PRs unchecked. Add a GitHub Actions workflow that runs swift
test with coverage (uploaded to Codecov) and SwiftLint.

- Bump macOS platform minimum to 10.15: the async/await public API
  already requires it, and the os_log(_:log:_:_:) call in
  InternetArchive.swift needs 10.14+, which broke `swift test` under
  current toolchains
- Exclude .build from SwiftLint so dependency checkouts aren't linted
- Exempt reviewer_itemname from identifier_name (mirrors the IA API)
- Update README badges (Travis -> Actions, codecov master -> main)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…-f9481b

* origin/main:
  Surface archive.org HTTP-200 error bodies as apiError; assert on oversized q (#22)
The 171-char os_log format string from #22 is a StaticString, which
can't be split across lines, so suppress the lint rule for that line.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@f738aac). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...ternetArchiveKit/InternetArchiveURLGenerator.swift 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage        ?   88.35%           
=======================================
  Files           ?       13           
  Lines           ?      541           
  Branches        ?        0           
=======================================
  Hits            ?      478           
  Misses          ?       63           
  Partials        ?        0           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jbuckner
jbuckner merged commit 33b1640 into main Jun 19, 2026
6 checks passed
@jbuckner
jbuckner deleted the claude/serene-neumann-f9481b branch June 19, 2026 17:59
jbuckner added a commit that referenced this pull request Jun 19, 2026
…ac33

* origin/main:
  Replace dead Travis CI with GitHub Actions (#24)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant