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 metadata for releasing w/project renames #637

Merged
merged 7 commits into from
Nov 20, 2023

Conversation

ashleygwilliams
Copy link
Contributor

No description provided.

bottomless-cli/Cargo.toml Outdated Show resolved Hide resolved
@ashleygwilliams ashleygwilliams force-pushed the cargo-dist-fix branch 4 times, most recently from cce7f7f to 09ebc14 Compare November 16, 2023 20:12
Copy link
Contributor

@MarinPostma MarinPostma left a comment

Choose a reason for hiding this comment

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

I don't see the old insta snapshot files being removed, I think the rename left them behind?

@ashleygwilliams
Copy link
Contributor Author

oh huh @MarinPostma that looks correct actually. i'll clean that up. additionally, there's a test i can't run locally due to a lack of a runtime dep i think- so that's the origin of the test failure i believe. will try to sort that but welcome any pointers!

@ashleygwilliams
Copy link
Contributor Author

(i'll also manage the conflict)

@MarinPostma
Copy link
Contributor

I fixed the test for you @ashleygwilliams, apparently there were just two missing snapshots 👍

@ashleygwilliams
Copy link
Contributor Author

(hopping in a call now but will fix my rebase botch when i'm out)

@MarinPostma MarinPostma added this pull request to the merge queue Nov 20, 2023
@MarinPostma
Copy link
Contributor

Thanks for the effort @ashleygwilliams 😄

Merged via the queue into tursodatabase:main with commit b0c05e0 Nov 20, 2023
12 checks passed
@ashleygwilliams ashleygwilliams deleted the cargo-dist-fix branch November 20, 2023 15:26
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.

3 participants