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

Drop Swift versions earlier than 5.8 #299

Merged
merged 5 commits into from
May 16, 2024
Merged

Drop Swift versions earlier than 5.8 #299

merged 5 commits into from
May 16, 2024

Conversation

gjcairo
Copy link
Contributor

@gjcairo gjcairo commented May 15, 2024

This PR removes support for Swift versions earlier than 5.8

Motivation:

Following the changes done in swift-metrics on apple/swift-metrics#141, I believe we should align swift-log with the rest of the ecosystem too and support only the last 3 Swift releases.

Modifications:

This PR removes support for Swift versions earlier than 5.8

Result:

Only the last 3 Swift releases are now supported.

@gjcairo gjcairo requested a review from ktoso May 15, 2024 12:57
Copy link
Member

@ktoso ktoso left a comment

Choose a reason for hiding this comment

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

Approved but may need to adjust CI right?

@ktoso
Copy link
Member

ktoso commented May 15, 2024

@swift-server-bot test this please

@ktoso
Copy link
Member

ktoso commented May 15, 2024

Mind checking the soundness (formatting issue)? The CI jobs for old swift we'll remove.

@FranzBusch
Copy link
Member

I think the soudness job just needs to be updated to use a newer docker file

@gjcairo
Copy link
Contributor Author

gjcairo commented May 15, 2024

No, there was an actual whitespace formatting issue - just fixed it.
The pipeline for api breakage has to be updated to use a newer docker image though. I've asked @yim-lee to take care of the CI changes.

@yim-lee
Copy link
Member

yim-lee commented May 15, 2024

CI updated. @swift-server-bot test this please.

@gjcairo
Copy link
Contributor Author

gjcairo commented May 15, 2024

@swift-server-bot test this please

@gjcairo
Copy link
Contributor Author

gjcairo commented May 15, 2024

Well, I don't think that worked :D @ktoso / @yim-lee mind triggering the tests again?

@FranzBusch
Copy link
Member

@swift-server-bot test this please

@FranzBusch
Copy link
Member

@swift-server-bot add to allowlist

@FranzBusch
Copy link
Member

@gjcairo We gotta update the breaking change checker script the same way we did in metrics.

@ktoso
Copy link
Member

ktoso commented May 15, 2024

branch protection rules adjusted in github -- can you fix the api checker?

@gjcairo gjcairo merged commit c7dcb55 into main May 16, 2024
6 checks passed
@gjcairo gjcairo deleted the drop-old-swift-versions branch May 16, 2024 09:11
ayushi2103 pushed a commit to ayushi2103/swift-log that referenced this pull request May 17, 2024
* Drop Swift versions earlier than 5.8

* Delete build_podspec.sh

* Update check_no_api_breakages script
cgrindel-self-hosted-renovate bot added a commit to cgrindel/rules_swift_package_manager that referenced this pull request Jun 27, 2024
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [apple/swift-log](https://togithub.com/apple/swift-log) | minor |
`from: "1.5.4"` -> `from: "1.6.1"` |

---

### Release Notes

<details>
<summary>apple/swift-log (apple/swift-log)</summary>

### [`v1.6.1`](https://togithub.com/apple/swift-log/releases/tag/1.6.1):
Swift Log 1.6.1

[Compare
Source](https://togithub.com/apple/swift-log/compare/1.6.0...1.6.1)

##### SemVer Patch

- Disable existential any build setting
([#&#8203;312](https://togithub.com/apple/swift-log/issues/312))

### [`v1.6.0`](https://togithub.com/apple/swift-log/releases/tag/1.6.0)

[Compare
Source](https://togithub.com/apple/swift-log/compare/1.5.4...1.6.0)

#### SemVer Minor

- Add Sendability annotations in
[apple/swift-log#308
- Fix deprecation warnings around default log implementations on
handlers in
[apple/swift-log#310
- Drop Swift versions earlier than 5.8 in
[apple/swift-log#299
- Implement Copy-On-Write (CoW) behavior for Logger struct by
[@&#8203;ayushi2103](https://togithub.com/ayushi2103) in
[apple/swift-log#297

##### SemVer Patch

- Replace standardOutput to standardError by
[@&#8203;ayushi2103](https://togithub.com/ayushi2103) in
[apple/swift-log#295
- Use Set to spot duplicated log handler warnings in
[apple/swift-log#306
- Make protocol usage obvious using any and some keywords in
[apple/swift-log#307
- Remove documentation for non-existent arguments by
[@&#8203;b1ackturtle](https://togithub.com/b1ackturtle) in
[apple/swift-log#309
- Remove Docc plugin which is no longer required in
[apple/swift-log#311

##### Other Changes

- Remove archived repository in
[apple/swift-log#292
- Add CI for Swift 5.10 in
[apple/swift-log#287
- Added swift-log-ecs to README.md by
[@&#8203;rwbutler](https://togithub.com/rwbutler) in
[apple/swift-log#298
- Update README.md add shipbook as backend by
[@&#8203;elishas](https://togithub.com/elishas) in
[apple/swift-log#304

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://togithub.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMDkuNCIsInVwZGF0ZWRJblZlciI6IjM2LjEwOS40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: Self-hosted Renovate Bot <361546+cgrindel-self-hosted-renovate[bot]@users.noreply.github.enterprise.com>
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.

None yet

4 participants