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 .spi.yml #785

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Update .spi.yml #785

merged 2 commits into from
Oct 6, 2023

Conversation

finestructure
Copy link
Contributor

Fix watchOS builds by selecting correct scheme

Fix watchOS builds by selecting correct scheme
@stephencelis
Copy link
Member

@finestructure Just to understand, does this mean watchOS isn't building InlineSnapshotTesting?

@finestructure
Copy link
Contributor Author

It wouldn't, no. We only support a single scheme and we've got to pick one here. Unless the repo came with an xcodeproj file that sets up a different scheme that builds both of the other schemes (or targets, in SwiftPM parlance).

It really comes down to which target/scheme you prefer as a compatibility indicator.

Actually, I've just looked at the package manifest and InlineSnapshotTesting depends on SnapshotTesting. So the easiest thing is just to use InlineSnapshotTesting as the "driving" scheme. I'll update the PR.

Copy link
Member

@stephencelis stephencelis left a comment

Choose a reason for hiding this comment

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

Thanks!

@stephencelis stephencelis merged commit c648dca into pointfreeco:main Oct 6, 2023
3 checks passed
thestephenmarshall referenced this pull request in powerhome/playbook-swift Oct 20, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence | Type |
Update |
|---|---|---|---|---|---|---|---|
| [fastlane](https://fastlane.tools)
([source](https://togithub.com/fastlane/fastlane),
[changelog](https://togithub.com/fastlane/fastlane/releases)) | `"~>
2.213.0"` -> `"~> 2.216.0"` |
[![age](https://developer.mend.io/api/mc/badges/age/rubygems/fastlane/2.216.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/rubygems/fastlane/2.216.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/rubygems/fastlane/2.213.0/2.216.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/rubygems/fastlane/2.213.0/2.216.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| | minor |
|
[nicklockwood/SwiftFormat](https://togithub.com/nicklockwood/SwiftFormat)
| `from: "0.52.6"` -> `from: "0.52.8"` |
[![age](https://developer.mend.io/api/mc/badges/age/git-tags/nicklockwood%2fSwiftFormat/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/git-tags/nicklockwood%2fSwiftFormat/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/git-tags/nicklockwood%2fSwiftFormat/0.52.6/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/git-tags/nicklockwood%2fSwiftFormat/0.52.6/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| | patch |
|
[nicklockwood/SwiftFormat](https://togithub.com/nicklockwood/SwiftFormat)
| `0.52.6` -> `0.52.8` |
[![age](https://developer.mend.io/api/mc/badges/age/github-tags/nicklockwood%2fSwiftFormat/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/github-tags/nicklockwood%2fSwiftFormat/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/github-tags/nicklockwood%2fSwiftFormat/0.52.6/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/github-tags/nicklockwood%2fSwiftFormat/0.52.6/0.52.8?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| repository | patch |
|
[pointfreeco/swift-snapshot-testing](https://togithub.com/pointfreeco/swift-snapshot-testing)
| `from: "1.14.0"` -> `from: "1.14.2"` |
[![age](https://developer.mend.io/api/mc/badges/age/git-tags/pointfreeco%2fswift-snapshot-testing/1.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/git-tags/pointfreeco%2fswift-snapshot-testing/1.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/git-tags/pointfreeco%2fswift-snapshot-testing/1.14.0/1.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/git-tags/pointfreeco%2fswift-snapshot-testing/1.14.0/1.14.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| | patch |
|
[pre-commit/pre-commit-hooks](https://togithub.com/pre-commit/pre-commit-hooks)
| `v4.4.0` -> `v4.5.0` |
[![age](https://developer.mend.io/api/mc/badges/age/github-tags/pre-commit%2fpre-commit-hooks/v4.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/github-tags/pre-commit%2fpre-commit-hooks/v4.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/github-tags/pre-commit%2fpre-commit-hooks/v4.4.0/v4.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/github-tags/pre-commit%2fpre-commit-hooks/v4.4.0/v4.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| repository | minor |

Note: The `pre-commit` manager in Renovate is not supported by the
`pre-commit` maintainers or community. Please do not report any problems
there, instead [create a Discussion in the Renovate
repository](https://togithub.com/renovatebot/renovate/discussions/new)
if you have any questions.

---

### Release Notes

<details>
<summary>fastlane/fastlane (fastlane)</summary>

###
[`v2.216.0`](https://togithub.com/fastlane/fastlane/releases/tag/2.216.0):
Fix for match GitLab storage

[Compare
Source](https://togithub.com/fastlane/fastlane/compare/2.215.1...2.216.0)

- \[match] fixes missing job-token error in GitLab storage mode
([#&#8203;21520](https://togithub.com/fastlane/fastlane/issues/21520))
via Darby Frey ([@&#8203;darbyfrey](https://togithub.com/darbyfrey))

###
[`v2.215.1`](https://togithub.com/fastlane/fastlane/releases/tag/2.215.1):
Workaround for Google Play API instability

[Compare
Source](https://togithub.com/fastlane/fastlane/compare/2.215.0...2.215.1)

- \[supply] add new SUPPLY_UPLOAD_MAX_RETRIES env var to attempt to
solve failed Google API calls
([#&#8203;21518](https://togithub.com/fastlane/fastlane/issues/21518))
via Josh Holtz ([@&#8203;joshdholtz](https://togithub.com/joshdholtz))

###
[`v2.215.0`](https://togithub.com/fastlane/fastlane/releases/tag/2.215.0):
Improvements

[Compare
Source](https://togithub.com/fastlane/fastlane/compare/2.214.0...2.215.0)

- \[spaceship] add request retry with backoff when encountring 429
errors
([#&#8203;21511](https://togithub.com/fastlane/fastlane/issues/21511))
via Stefan Natchev ([@&#8203;snatchev](https://togithub.com/snatchev))
- \[supply] introduce a new synchronization logic for screenshots
([#&#8203;21498](https://togithub.com/fastlane/fastlane/issues/21498))
via Olivier Halligon
([@&#8203;AliSoftware](https://togithub.com/AliSoftware))
- \[deliver] validate the status of app version to be ready_for_review
before submitting the app
([#&#8203;20515](https://togithub.com/fastlane/fastlane/issues/20515))
via 醉 ([@&#8203;404pilot](https://togithub.com/404pilot))
- \[match] Added s3\_skip_encryption parameter
([#&#8203;21018](https://togithub.com/fastlane/fastlane/issues/21018))
via Morten Bøgh ([@&#8203;mbogh](https://togithub.com/mbogh))
- \[docs] add minor branding guidelines to `CONTRIBUTING.md`
([#&#8203;21495](https://togithub.com/fastlane/fastlane/issues/21495))
via Roger Oba ([@&#8203;rogerluan](https://togithub.com/rogerluan))
- \[match] prevent directory download from s3
([#&#8203;20975](https://togithub.com/fastlane/fastlane/issues/20975))
via Mark Homoki ([@&#8203;markhomoki](https://togithub.com/markhomoki))
- \[xcodes] fix issue where `xcodes` action wouldn't accept beta
versions of Xcode
([#&#8203;21434](https://togithub.com/fastlane/fastlane/issues/21434))
via apps4everyone
([@&#8203;apps4everyone](https://togithub.com/apps4everyone))
- \[spaceship] explicit beta group access to all builds at creation
([#&#8203;21478](https://togithub.com/fastlane/fastlane/issues/21478))
via Vincent Isambart
([@&#8203;vincentisambart](https://togithub.com/vincentisambart))
- \[docs] update pull request template to ask contributors to write unit
tests
([#&#8203;21427](https://togithub.com/fastlane/fastlane/issues/21427))
via Roger Oba ([@&#8203;rogerluan](https://togithub.com/rogerluan))
- \[action] deprecate the ensure_xcode_version action
([#&#8203;21448](https://togithub.com/fastlane/fastlane/issues/21448))
via Iulian Onofrei ([@&#8203;revolter](https://togithub.com/revolter))
- \[tests] stop generating rspec temp file in current directory
([#&#8203;21466](https://togithub.com/fastlane/fastlane/issues/21466))
via Roger Oba ([@&#8203;rogerluan](https://togithub.com/rogerluan))
- \[ci] fix CI by replacing deprecated Xcode versions, and add Xcode 14,
Xcode 15, and Ruby 3.1 to test envs
([#&#8203;21465](https://togithub.com/fastlane/fastlane/issues/21465))
via Roger Oba ([@&#8203;rogerluan](https://togithub.com/rogerluan))
- \[deliver]\[frameit] add iPhone 14 Pro / 6.1" display type
([#&#8203;21393](https://togithub.com/fastlane/fastlane/issues/21393))
via Stefan Natchev ([@&#8203;snatchev](https://togithub.com/snatchev))
- \[spaceship] add missing `AppStoreState` cases
([#&#8203;21312](https://togithub.com/fastlane/fastlane/issues/21312))
via Roger Oba ([@&#8203;rogerluan](https://togithub.com/rogerluan))
- \[fastlane_core] fix WWDR certificates import flow
([#&#8203;21442](https://togithub.com/fastlane/fastlane/issues/21442))
via Paul Taykalo
([@&#8203;PaulTaykalo](https://togithub.com/PaulTaykalo))
- \[docs] fix tool name formatting (for `supply`) to be lowercase to
pass fastlane docs unit tests
([#&#8203;21426](https://togithub.com/fastlane/fastlane/issues/21426))
via Roger Oba ([@&#8203;rogerluan](https://togithub.com/rogerluan))

###
[`v2.214.0`](https://togithub.com/fastlane/fastlane/releases/tag/2.214.0):
Improvements

[Compare
Source](https://togithub.com/fastlane/fastlane/compare/2.213.0...2.214.0)

- \[spaceship] fix olympus session request error after skipping 2FA
Upgrade
([#&#8203;21317](https://togithub.com/fastlane/fastlane/issues/21317))
via AbbyM ([@&#8203;AbbyM](https://togithub.com/AbbyM))
- \[spaceship] add update method to ConnectAPI::User
([#&#8203;20956](https://togithub.com/fastlane/fastlane/issues/20956))
via Vitalii Budnik ([@&#8203;nekrich](https://togithub.com/nekrich))
- \[supply]\[action] add `download_apk_from_google_play` action (and
corresponding `supply` methods)
([#&#8203;21315](https://togithub.com/fastlane/fastlane/issues/21315))
via Olivier Halligon
([@&#8203;AliSoftware](https://togithub.com/AliSoftware))
- \[docs] fix sync_code_signing.md
([#&#8203;21202](https://togithub.com/fastlane/fastlane/issues/21202))
via Yuki Sakai ([@&#8203;VeyronSakai](https://togithub.com/VeyronSakai))
- \[fastlane] fix fastlane action template warnings reported by Rubocop
([#&#8203;21310](https://togithub.com/fastlane/fastlane/issues/21310))
via Bartosz Kosiorek
([@&#8203;BartoszKosiorek-TomTom](https://togithub.com/BartoszKosiorek-TomTom))
- \[docs]\[match] fix typos, improve wording, incorporate improvements
to match documentation examples
([#&#8203;21302](https://togithub.com/fastlane/fastlane/issues/21302))
via Roger Oba ([@&#8203;rogerluan](https://togithub.com/rogerluan))
- \[match] fixes Gitlab Secure Files API limit
([#&#8203;21361](https://togithub.com/fastlane/fastlane/issues/21361))
via Darby Frey ([@&#8203;darbyfrey](https://togithub.com/darbyfrey))
- \[match] adding access token prompt for GitLab Secure Files Match
([#&#8203;21279](https://togithub.com/fastlane/fastlane/issues/21279))
via Darby Frey ([@&#8203;darbyfrey](https://togithub.com/darbyfrey))
- \[deliver] update upload_price_tier
([#&#8203;21282](https://togithub.com/fastlane/fastlane/issues/21282))
via lukethoene ([@&#8203;lukethoene](https://togithub.com/lukethoene))

</details>

<details>
<summary>nicklockwood/SwiftFormat (nicklockwood/SwiftFormat)</summary>

###
[`v0.52.8`](https://togithub.com/nicklockwood/SwiftFormat/blob/HEAD/CHANGELOG.md#0528-2023-10-17)

[Compare
Source](https://togithub.com/nicklockwood/SwiftFormat/compare/0.52.7...0.52.8)

- Fixed `redundantClosure` rule in cases where an `if`/`switch`
expression is not permitted
-   The `docComments` rule now correctly handles macro comments
- The `docComments` rule is now only applied to a comment block if all
lines match the pattern

###
[`v0.52.7`](https://togithub.com/nicklockwood/SwiftFormat/blob/HEAD/CHANGELOG.md#0527-2023-10-06)

[Compare
Source](https://togithub.com/nicklockwood/SwiftFormat/compare/0.52.6...0.52.7)

- Fixed bug where `conditionalAssignment` and `redundantClosure` rules
would be applied incorrectly
- Fixed `redundantClosure` rule leaving stray `try` or `await` keywords
behind

</details>

<details>
<summary>pointfreeco/swift-snapshot-testing
(pointfreeco/swift-snapshot-testing)</summary>

###
[`v1.14.2`](https://togithub.com/pointfreeco/swift-snapshot-testing/releases/tag/1.14.2)

[Compare
Source](https://togithub.com/pointfreeco/swift-snapshot-testing/compare/1.14.1...1.14.2)

#### What's Changed

- Fixed: Inline snapshots are now prepared *before* emitting an XCTest
failure, ensuring the artifact is written
([https://github.com/pointfreeco/swift-snapshot-testing/pull/787](https://togithub.com/pointfreeco/swift-snapshot-testing/pull/787)).
- Fixed: `assertCustomInline()` no longer leaves empty parens hanging
around when expanded
([https://github.com/pointfreeco/swift-snapshot-testing/pull/788](https://togithub.com/pointfreeco/swift-snapshot-testing/pull/788)).
- Fixed: The `swift-syntax` dependency has widened to support
508.0.1..<510.0.0, allowing the library to be depended on even when
another package depends on 508.0.1
([https://github.com/pointfreeco/swift-snapshot-testing/pull/795](https://togithub.com/pointfreeco/swift-snapshot-testing/pull/795)).
- Infrastructure: Updated SPI configuration for watchOS (thanks
[@&#8203;finestructure](https://togithub.com/finestructure),
[https://github.com/pointfreeco/swift-snapshot-testing/pull/789](https://togithub.com/pointfreeco/swift-snapshot-testing/pull/789)).

**Full Changelog**:
pointfreeco/swift-snapshot-testing@1.14.1...1.14.2

###
[`v1.14.1`](https://togithub.com/pointfreeco/swift-snapshot-testing/releases/tag/1.14.1)

[Compare
Source](https://togithub.com/pointfreeco/swift-snapshot-testing/compare/1.14.0...1.14.1)

#### What's Changed

- Fixed: Avoid inline snapshot trailing closure index crash
([https://github.com/pointfreeco/swift-snapshot-testing/pull/786](https://togithub.com/pointfreeco/swift-snapshot-testing/pull/786)).
- Infrastructure: Update .spi.yml to show watchOS support (thanks
[@&#8203;finestructure](https://togithub.com/finestructure),
[https://github.com/pointfreeco/swift-snapshot-testing/pull/785](https://togithub.com/pointfreeco/swift-snapshot-testing/pull/785)).

**Full Changelog**:
pointfreeco/swift-snapshot-testing@1.14.0...1.14.1

</details>

<details>
<summary>pre-commit/pre-commit-hooks
(pre-commit/pre-commit-hooks)</summary>

###
[`v4.5.0`](https://togithub.com/pre-commit/pre-commit-hooks/compare/v4.4.0...v4.5.0)

[Compare
Source](https://togithub.com/pre-commit/pre-commit-hooks/compare/v4.4.0...v4.5.0)

</details>

---

### Configuration

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

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, 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 [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/powerhome/PlaybookSwift).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xOS4yIiwidXBkYXRlZEluVmVyIjoiMzcuMTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Stephen Marshall <smarshall1980@gmail.com>
Muhieddine-El-Kaissi pushed a commit to thumbtack/swift-snapshot-testing that referenced this pull request Aug 8, 2024
* Update .spi.yml

Fix watchOS builds by selecting correct scheme

* Update .spi.yml
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