Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

Prepare for release 0.0.2 #35

Merged
merged 2 commits into from
Apr 19, 2022
Merged

Prepare for release 0.0.2 #35

merged 2 commits into from
Apr 19, 2022

Conversation

bfish713
Copy link
Contributor

@bfish713 bfish713 commented Apr 15, 2022

Description

Preparing a release for Seahorse to pin to. This just updates the cargo.toml to match what I plan to tag as the release.


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Linked to GitHub issue with discussion and accepted design OR have an explanation in the PR that describes this work.
  • Wrote unit tests - N/A
  • Updated relevant documentation in the code - N/A
  • Added a relevant changelog entry to the Pending section in CHANGELOG.md - N/A
  • Re-reviewed Files changed in the GitHub PR explorer

@bfish713 bfish713 marked this pull request as ready for review April 18, 2022 18:10
@alxiong
Copy link
Contributor

alxiong commented Apr 19, 2022

Wait, why aren't we tagging 0.1.1? @bfish713

@bfish713
Copy link
Contributor Author

Wait, why aren't we tagging 0.1.1? @bfish713

Oh it's a bit confusing, 0.1.1 will be the next version for jellyfish. We have to use 0.1.0 here because this PR EspressoSystems/jellyfish#57 seems have some breaking changes. So I can't use the most recent version of jellyfish. Let me know if that doesn't sound right

@alxiong
Copy link
Contributor

alxiong commented Apr 19, 2022

We have to use 0.1.0 here because this PR EspressoSystems/jellyfish#57 seems have some breaking changes. So I can't use the most recent version of jellyfish.

I think issue 57 was mostly internal API changes, shouldn't affect downstream (at least not too much)? so did you try to use 0.1.1 and run into a lot of compiler err?


in any case, I think it's fine to use 0.1.0 for now. feel free to merge it.

@bfish713
Copy link
Contributor Author

We have to use 0.1.0 here because this PR EspressoSystems/jellyfish#57 seems have some breaking changes. So I can't use the most recent version of jellyfish.

I think issue 57 was mostly internal API changes, shouldn't affect downstream (at least not too much)? so did you try to use 0.1.1 and run into a lot of compiler err?

in any case, I think it's fine to use 0.1.0 for now. feel free to merge it.

Yeah was having issues compiling using 0.1.1

@bfish713 bfish713 merged commit b64e513 into main Apr 19, 2022
@bfish713 bfish713 deleted the release-0.0.2 branch April 19, 2022 14:00
jbearer added a commit that referenced this pull request May 19, 2022
* Prepare for release 0.0.2 (#35)

* Prepare for release 0.0.2

* Update jellyfish tag

* Prepare for release 0.0.3 (#38)

* Prepare for release 0.0.3

* Updates for latest version of Jellyfish

Co-authored-by: bfish713 <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants