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

FIX Update to latest, including Swift Testing support #6

Merged
merged 11 commits into from
Dec 10, 2024

Conversation

mblackmon
Copy link
Collaborator

@mblackmon mblackmon commented Dec 9, 2024

Update to latest to get the Swift Testing changes.

Related: GUAC-5510

mbrandonw and others added 11 commits August 1, 2024 15:13
It is no longer possible to `@_spi(Experimental) import Testing`, so we
can no longer ship a custom execution test trait.

Instead of using such a trait to override a task local for the duration
of a test, we can use a more general trait to store the information, and
the assertion helpers can coalesce to the trait, instead.
* fix formatting of numbers to have stable snapshots

as seen in pointfreeco#533

* fix to make tests compile

pointfreeco#887

---------

Co-authored-by: stijn <[email protected]>
* fix missing inline snapshot testing

* remove the need to import inline in not inline snapshot target

---------

Co-authored-by: stijn <[email protected]>
* Fix assertSnapshot for Swift Testing tests.

* wip

* Update CI
@lj-dickey
Copy link

It seems like the newest version of main in our fork is 1.17.2, but 1.17.6 was released in October... should be be using that newer version?

@mblackmon
Copy link
Collaborator Author

I think this is 1.17.6? The presentation on GitHub is a bit confusing to me and makes me thing the latest commit to this branch was 5 months ago. However, when I pull this down locally, my log looks like this:

* 42a0861 - (9 weeks ago) Run swift-format — mbrandonw (HEAD -> fix/update-dec-2024, tag: 1.17.6, upstream/main, origin/fix/update-dec-2024)

In case it's not obvious, I'm shooting to update to 1.17.6

@mblackmon mblackmon merged commit 7c06008 into main Dec 10, 2024
4 checks passed
@mblackmon mblackmon deleted the fix/update-dec-2024 branch December 10, 2024 18:07
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.

5 participants