Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update DocC for rules_swift 2.1.0 #2487

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

luispadron
Copy link
Contributor

@luispadron luispadron commented Jul 5, 2024

Updates the DocC rules to support rules_swift 2.x which introduced breaking changes to how Swift symbol graphs are collected.

This PR updates to use the new aspect to extract the symbol graphs. The public API remains unchanged.

Fixes #2445

@luispadron luispadron changed the title WIP: update docc rules for swift 2 WIP: update docc rules for rules_swift 2 Jul 5, 2024
@luispadron luispadron force-pushed the luis/update-docc-rules-for-swift-2 branch 3 times, most recently from 6cc3eb9 to 5d6214f Compare July 5, 2024 19:17
@luispadron luispadron force-pushed the luis/update-to-rules_swift-2 branch 2 times, most recently from a8e21cd to f56bb01 Compare July 9, 2024 14:09
@luispadron luispadron mentioned this pull request Jul 9, 2024
1 task
@luispadron luispadron force-pushed the luis/update-to-rules_swift-2 branch from f56bb01 to c1b44bf Compare July 9, 2024 14:20
brentleyjones pushed a commit that referenced this pull request Jul 9, 2024
Updates the **rules_swift** dependency to 2.0.0. This unblocks the new
rules/aspects in other planned PRs

## Tasks

- [x] Wait for a final 2.0.0 release

## Follow up tasks

- Merge #2418
- Merge #2487
Base automatically changed from luis/update-to-rules_swift-2 to master July 9, 2024 14:40
@luispadron luispadron force-pushed the luis/update-docc-rules-for-swift-2 branch 2 times, most recently from be85b8d to 4fb192e Compare July 9, 2024 15:06
Updates the DocC rules to support rules_swift 2.x which introduced breaking changes to how Swift symbol graphs are collected.

This PR updates to use the new aspect to extract the symbol graphs. The public API remains unchanged.
@luispadron luispadron force-pushed the luis/update-docc-rules-for-swift-2 branch 3 times, most recently from 7f26425 to 168a214 Compare July 9, 2024 18:17
@luispadron luispadron changed the title WIP: update docc rules for rules_swift 2 Update DocC for rules_swift 2.1.0 Jul 9, 2024
@luispadron luispadron marked this pull request as ready for review July 9, 2024 18:18
@luispadron luispadron force-pushed the luis/update-docc-rules-for-swift-2 branch from 168a214 to 2f42112 Compare July 9, 2024 18:19
@luispadron luispadron force-pushed the luis/update-docc-rules-for-swift-2 branch from 2f42112 to 3b0c933 Compare July 9, 2024 18:30
apple/internal/aspects/docc_archive_aspect.bzl Outdated Show resolved Hide resolved
apple/internal/aspects/docc_archive_aspect.bzl Outdated Show resolved Hide resolved
@luispadron luispadron force-pushed the luis/update-docc-rules-for-swift-2 branch 2 times, most recently from 7acadd3 to ebffa7c Compare July 10, 2024 14:04
@brentleyjones brentleyjones merged commit 8e33d08 into master Jul 10, 2024
7 of 11 checks passed
@brentleyjones brentleyjones deleted the luis/update-docc-rules-for-swift-2 branch July 10, 2024 16:12
brentleyjones pushed a commit that referenced this pull request Jul 10, 2024
Updating to rules_swift 2.x will be required by:

- #2418 
- #2487
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.

Support extension blocks in DocC rule
2 participants