Skip to content

Conversation

rauhul
Copy link
Contributor

@rauhul rauhul commented Feb 6, 2025

Adds CI to check for valid formatting and a variety of other checks.

@rauhul
Copy link
Contributor Author

rauhul commented Feb 6, 2025

@swift-ci test

Comment on lines 20 to 23
- name: Compare against swift-mmio swift-format config
run: |
curl -sL https://raw.githubusercontent.com/apple/swift-mmio/refs/heads/main/.swift-format -o .swift-format-mmio
diff .swift-format .swift-format-mmio
Copy link
Member

Choose a reason for hiding this comment

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

Why are we comparing with swift-mmio repo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dropped this for now

push:
branches: ["main"]
pull_request:
branches: ["main"]
Copy link
Member

Choose a reason for hiding this comment

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

We should also add

types: [opened, reopened, synchronize]

Recommend checking out https://github.com/apple/swift-collections/pull/426/files

Copy link
Contributor Author

Choose a reason for hiding this comment

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

switched to match collections

Adds CI to check for valid formatting and a variety of other checks.
@rauhul
Copy link
Contributor Author

rauhul commented Feb 7, 2025

@swift-ci test

@rauhul rauhul enabled auto-merge (squash) February 7, 2025 19:23
@rauhul rauhul merged commit 33acc79 into main Feb 7, 2025
11 checks passed
@rauhul rauhul deleted the soundness branch February 7, 2025 19: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.

2 participants