Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@stonko1994
Copy link
Contributor

@stonko1994 stonko1994 commented Nov 3, 2023

Fixes #1246

Description

This PR adds visionOS support to Quick 🥳

Before this PR can be merged, the counterpart PR in Nimble needs to be merged:

Checklist - While not every PR needs it, new features should consider this list:

  • Does this have tests?
  • Does this have documentation?
  • Does this break the public API (Requires major version bump)?
  • Is this a new feature (Requires minor version bump)?

@stonko1994 stonko1994 force-pushed the bitmovin/feature/add-visionos-support branch from 94b60c6 to a4e42fd Compare November 3, 2023 10:19
@stonko1994 stonko1994 force-pushed the bitmovin/feature/add-visionos-support branch from 5331a92 to 487c837 Compare January 19, 2024 15:56
@stonko1994 stonko1994 marked this pull request as ready for review January 19, 2024 15:58
@stonko1994
Copy link
Contributor Author

@younata Since the Nimble PR was merged and a new version was release, this PR is now also ready for review. Please execute the CI to see if everything still works. (I did not add a visionOS CI workflow yet due to the same reason as for Nimble)

desc "Run lint for podspec"
task :lint do
run "bundle exec pod lib lint"
run "bundle exec pod lib lint --platforms=ios,tvos,macos"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need to skip visionos for now as the CI does not yet have visionOS simulators installed. We did the same for Nimble and need to revisit this once available.

@stonko1994
Copy link
Contributor Author

@younata I 'fixed' the cocoapod lint step by skipping visionOS for now. However, I don't understand why the danger step is failing 👀 Do you have any insights there?

@younata
Copy link
Member

younata commented Jan 24, 2024

However, I don't understand why the danger step is failing 👀 Do you have any insights there?

It's because this PR is not from the Quick org. It's some permissions issue that I haven't thought to figure out. Don't worry about it. This s good to merge.

@younata younata merged commit d709425 into Quick:main Jan 24, 2024
@younata
Copy link
Member

younata commented Jan 24, 2024

And included in Quick 7.4.0! Thanks! https://github.com/Quick/Quick/releases/tag/v7.4.0

cgrindel-self-hosted-renovate bot referenced this pull request in cgrindel/rules_swift_package_manager Jan 24, 2024
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [Quick/Quick](https://togithub.com/Quick/Quick) | minor | `from:
"7.3.1"` -> `from: "7.4.0"` |

---

### Release Notes

<details>
<summary>Quick/Quick (Quick/Quick)</summary>

### [`v7.4.0`](https://togithub.com/Quick/Quick/releases/tag/v7.4.0): -
visionOS support

[Compare
Source](https://togithub.com/Quick/Quick/compare/v7.3.1...v7.4.0)

### Highlights

- Quick now supports visionOS! Thanks
[@&#8203;stonko1994](https://togithub.com/stonko1994)!

Note: This version of Quick is not yet in Cocoapods, see
[https://github.com/CocoaPods/CocoaPods/issues/12118](https://togithub.com/CocoaPods/CocoaPods/issues/12118)

### Autogenerated Release Notes

#### What's Changed

- Release script should auto-update local ruby dependencies by
[@&#8203;younata](https://togithub.com/younata) in
[https://github.com/Quick/Quick/pull/1263](https://togithub.com/Quick/Quick/pull/1263)
- Add visionOS support by
[@&#8203;stonko1994](https://togithub.com/stonko1994) in
[https://github.com/Quick/Quick/pull/1249](https://togithub.com/Quick/Quick/pull/1249)

#### New Contributors

- [@&#8203;stonko1994](https://togithub.com/stonko1994) made their first
contribution in
[https://github.com/Quick/Quick/pull/1249](https://togithub.com/Quick/Quick/pull/1249)

**Full Changelog**:
Quick/Quick@v7.3.1...v7.4.0

</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:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMDAuMCIsInVwZGF0ZWRJblZlciI6IjM2LjEwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

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.

Add visionOS support

2 participants