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
Rename all the things from libra to diem #6798
Conversation
libra.org -> diem.com, diem.org doesn't exist. |
5def085
to
a37da80
Compare
c80b5e4
to
f0c5cd2
Compare
f0c5cd2
to
13d05ca
Compare
use libra_network_address::NetworkAddress; | ||
use libra_types::chain_id::ChainId; | ||
use diem_network_address::NetworkAddress; | ||
use diem_types::chain_id::ChainId; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JoshLind I had to keep the libra
and libra.org
names below for the network addreses. Just want to make sure this is OK/that you're aware.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tzakian, this should be fine -- we can clean it up later 😄 Out of curiosity, did the test fail if you changed them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, sadly the test failed if they are changed :( So figured best to switch them back for now :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks! 😄
13d05ca
to
ced9527
Compare
ced9527
to
b4d380b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments for things that can be done later if this is good to go.
c077b7b
to
d03eaea
Compare
/cancel |
❗ Land has been canceled due to this PR being updated with new commits. Please issue another Land command if you want to requeue this PR. |
/land |
LEC dependency change summary:
|
release binaries dependency change summary:
|
LSR dependency change summary:
|
/cancel |
30172cd
to
831eb10
Compare
/land |
LSR dependency change summary:
|
LEC dependency change summary:
|
release binaries dependency change summary:
|
Hey hey! It looks like #wannabe-land-blocking found some issues that are not yet land blocking, but will be in the future. click here to reveal them
|
1 similar comment
Hey hey! It looks like #wannabe-land-blocking found some issues that are not yet land blocking, but will be in the future. click here to reveal them
|
Cluster Test Result
Repro cmd:
🎉 Land-blocking cluster test passed! 👌 |
831eb10
to
7657697
Compare
Hey hey! It looks like #wannabe-land-blocking found some issues that are not yet land blocking, but will be in the future. click here to reveal them
|
/// consists in this global prefix, concatenated with the specified | ||
/// serialization name of the struct. | ||
pub(crate) const LIBRA_HASH_PREFIX: &[u8] = b"LIBRA::"; | ||
pub(crate) const DIEM_HASH_PREFIX: &[u8] = b"DIEM::"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we still making hard backwards incompatible breaking changes? This is one of them...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, trying to do it all at once.
Renames all occurrences of "libra" with "diem" (with casing preserved).
Renames all occurrences of "lbr" with "XDX" (with casing preserved).
Renames all occurences of "coin1" to "XUS" (with casing preserved).
This PR is meant to be the bulk renaming, there will need to be smaller cleanup passes that we will want to do afterwards, but that can be done by folks in each area.