Skip to content

docs(readme): add Swift Package Index compatibility badges#49

Merged
youtalk merged 2 commits intomainfrom
readme-spi-badges
Apr 28, 2026
Merged

docs(readme): add Swift Package Index compatibility badges#49
youtalk merged 2 commits intomainfrom
readme-spi-badges

Conversation

@youtalk
Copy link
Copy Markdown
Owner

@youtalk youtalk commented Apr 28, 2026

Summary

Current SPI matrix output:

  • swift-versions: `6.3 | 6.2 | 6.1 | 6.0`
  • platforms: `iOS | macOS | visionOS`

Test plan

The package was added to swiftpackageindex.com/youtalk/swift-ros2 via
PackageList PR #13355. SPI now publishes dynamic compatibility badges
derived from its build matrix:

- swift-versions: 6.3 | 6.2 | 6.1 | 6.0
- platforms: iOS | macOS | visionOS

Both link back to the SPI package page so readers can drill into the
full per-platform / per-Swift status and the generated DocC
documentation.
Copilot AI review requested due to automatic review settings April 28, 2026 13:55
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Swift Package Index (SPI) compatibility badges to the README so readers can quickly see supported Swift versions and platforms, with links to the SPI package page.

Changes:

  • Add SPI dynamic badge for Swift version compatibility.
  • Add SPI dynamic badge for platform compatibility.
  • Link both badges to the SPI package page for drill-down details and DocC.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment on lines +8 to +12
[![Swift](https://img.shields.io/badge/Swift-5.9%20%7C%206.0%20%7C%206.3-orange.svg)](https://swift.org)
[![ROS 2](https://img.shields.io/badge/ROS%202-Humble%20%7C%20Jazzy%20%7C%20Kilted%20%7C%20Rolling-22314E.svg)](https://docs.ros.org)
[![License](https://img.shields.io/badge/license-Apache%202.0-blue.svg)](LICENSE)
[![SPI Swift compatibility](https://img.shields.io/endpoint?url=https%3A%2F%2Fswiftpackageindex.com%2Fapi%2Fpackages%2Fyoutalk%2Fswift-ros2%2Fbadge%3Ftype%3Dswift-versions)](https://swiftpackageindex.com/youtalk/swift-ros2)
[![SPI platform compatibility](https://img.shields.io/endpoint?url=https%3A%2F%2Fswiftpackageindex.com%2Fapi%2Fpackages%2Fyoutalk%2Fswift-ros2%2Fbadge%3Ftype%3Dplatforms)](https://swiftpackageindex.com/youtalk/swift-ros2)
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The README now shows two Swift compatibility indicators with different version sets: the static Swift badge lists 5.9 | 6.0 | 6.3, while the new SPI badge will report the SPI-tested versions (per PR description: 6.0–6.3, including 6.1/6.2). This is confusing for readers; consider updating or removing the static Swift badge so the Swift version story is consistent with SPI/CI.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point — dropped the static 5.9 | 6.0 | 6.3 badge in 48489ab. SPI's dynamic badge (currently 6.3 | 6.2 | 6.1 | 6.0) is now the single source of truth for Swift compatibility, and the minimum version stays declared in Package.swift's swift-tools-version: 5.9 for anyone who needs to look it up.

Address PR review (Copilot, #49): the static Swift badge listed
\`5.9 | 6.0 | 6.3\` while the SPI dynamic swift-versions badge shows the
actually-tested set \`6.0 | 6.1 | 6.2 | 6.3\`. Two badges with different
version sets is more confusing than informative.

Drop the static badge and let SPI be the canonical source of Swift
compatibility — it auto-updates as the SPI matrix evolves and links
back to the per-version build status. Minimum Swift remains declared
in Package.swift (\`swift-tools-version: 5.9\`) for anyone who needs it.
@youtalk youtalk merged commit a28afc5 into main Apr 28, 2026
11 checks passed
@youtalk youtalk deleted the readme-spi-badges branch April 28, 2026 14:28
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.

2 participants