Skip to content

Fix README and podspec nits and run CI on main before 3.0.0 - #10

Merged
mabidakun merged 3 commits into
mainfrom
chore/pre-release-tidy
Oct 24, 2026
Merged

mabidakun merged 3 commits into
mainfrom
chore/pre-release-tidy

Conversation

@mabidakun

Copy link
Copy Markdown
Owner

What does this PR do?

  • Problem: three small faults remain before the 3.0.0 tag. The README usage line for withTextFilepath has a ) in the wrong place, so it does not show valid Swift. The README does not show that strings() with no prefix returns every string. The podspec starts with a blank line. Also, the default branch is now main, but CI still runs on pushes to master only, so merges get no CI run.
  • This PR:
    • moves the ) to the end of the withTextFilepath usage line and adds a strings() example (525ce1d).
    • removes the blank first line from the podspec (c4b869d).
    • runs CI on pushes to main (62f6da4). CI on pull requests does not change.
  • Risk: none for library users. No Swift source changes.

Test Evidence

Run locally on 62f6da4 with Xcode 27.1:

Check Command Result
Unit tests swift test 108 passed, 0 failed
Podspec lint pod lib lint --platforms=macos,ios passed
Workflow lint actionlint .github/workflows/ci.yml no issues
README example a temporary test of the new strings() example returned ["act", "apologise", "apology", "app", "apple"], as the README says

The CI run adds the full pod lib lint with tvOS.

Background

@mabidakun
mabidakun merged commit 4d9f9af into main Oct 24, 2026
1 check passed
@mabidakun
mabidakun deleted the chore/pre-release-tidy branch October 24, 2026 10:57
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