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

Frankie/core#v0.3.0.rc 1 #421

Merged
merged 67 commits into from
Oct 23, 2024
Merged

Frankie/core#v0.3.0.rc 1 #421

merged 67 commits into from
Oct 23, 2024

Conversation

frankiebee
Copy link
Collaborator

@frankiebee frankiebee commented Oct 8, 2024

This PR updates to the latest version of core (still in rc)

Copy link
Collaborator

@mixmix mixmix left a comment

Choose a reason for hiding this comment

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

This Review:

  • 🔥 couple gotchas / need change
  • ✔️ some console.log scrubbing
  • ⚠️ a TODO to remember for when -rc1 is removed
  • some style / requests (casual)

.github/workflows/test.yaml Outdated Show resolved Hide resolved
dev/README.md Outdated Show resolved Hide resolved
@@ -1,3 +1,3 @@
#! /usr/bin/env bash

export ENTROPY_CORE_VERSION=release-v0.2.0
export ENTROPY_CORE_VERSION=release-v0.3.0-rc.1
Copy link
Collaborator

@mixmix mixmix Oct 11, 2024

Choose a reason for hiding this comment

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

⚠️ TODO - change once new release

Suggested change
export ENTROPY_CORE_VERSION=release-v0.3.0-rc.1
export ENTROPY_CORE_VERSION=release-v0.3.0

# commit in the `entropy-core` repository. Refer to the wiki page at
# https://github.com/entropyxyz/meta/wiki/Git#conventions for details
# about how Git tags, branches, and commits refer to image tags here.
image: entropyxyz/entropy-tss:${ENTROPY_CORE_VERSION:-latest}
Copy link
Collaborator

Choose a reason for hiding this comment

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

🔥 I don't know if we want this to default to latest do we?

I 100% want to know what the version is I'm running and if I forgot to set the ENTROPY_CORE_VERSION this feels like I'm gonna get some random surprise version 👎

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this -latest came as a suggestion from @vitropy i believe i am hestant to remove this as i want to get the latest build for this version tag that i am asking for

Copy link
Contributor

Choose a reason for hiding this comment

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

@mixmix I think @frankiebee is correct here: if you 100% want to know what the version you are running is, simply set ENTROPY_CORE_VERSION in your environment. That is what this variable is for. See Environment variables in our Wiki. If you don't set it, then getting the latest tag mimics the behavior of all other Docker default commands, which is what 99% of users are going to want.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes I get this is a fallback. I just think it might be a foot-gun. Fine to wait and see though!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

lets wait and see

package.json Show resolved Hide resolved
tests/end-to-end.test.ts Outdated Show resolved Hide resolved
tests/end-to-end.test.ts Outdated Show resolved Hide resolved
tests/four-nodes.test.ts Show resolved Hide resolved
tests/four-nodes.test.ts Show resolved Hide resolved
tests/testing-utils/network.ts Outdated Show resolved Hide resolved
mixmix
mixmix previously approved these changes Oct 23, 2024
package.json Outdated Show resolved Hide resolved
@mixmix mixmix merged commit 8756dff into main Oct 23, 2024
3 checks passed
@mixmix mixmix deleted the frankie/core#v0.3.0.rc-1 branch October 23, 2024 20:58
@github-actions github-actions bot locked and limited conversation to collaborators Oct 23, 2024
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.

5 participants