test(ci): prepare apollo_starknet_os_program for publishing#5933
Conversation
d3aec12 to
80a3c36
Compare
0ce9251 to
fb39974
Compare
80a3c36 to
e825e2c
Compare
fb39974 to
4285720
Compare
e825e2c to
54559d9
Compare
4285720 to
bca7a50
Compare
54559d9 to
fef0fee
Compare
bca7a50 to
d36ae68
Compare
fef0fee to
d2bb6a1
Compare
d36ae68 to
1cd12cf
Compare
289c560 to
1b61458
Compare
e33ac04 to
992092c
Compare
1b61458 to
ec43aed
Compare
992092c to
8d21c22
Compare
ec43aed to
7c821de
Compare
8d21c22 to
cbb30a2
Compare
7c821de to
f78ef01
Compare
cbb30a2 to
b19279c
Compare
f78ef01 to
7802669
Compare
b19279c to
1b5c0a9
Compare
7802669 to
f2055cb
Compare
1b5c0a9 to
352577a
Compare
f2055cb to
2a4c18d
Compare
352577a to
ffb1f7f
Compare
2a4c18d to
7dbed53
Compare
ffb1f7f to
50bcaaa
Compare
7dbed53 to
ae5ecfb
Compare
50bcaaa to
0a438d2
Compare
ae5ecfb to
dee7200
Compare
0a438d2 to
02c9306
Compare
dee7200 to
80cdf40
Compare
02c9306 to
9161fb5
Compare
|
Benchmark movements: No major performance changes detected. |
TzahiTaub
left a comment
There was a problem hiding this comment.
Reviewed 1 of 2 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @amosStarkware and @nimrod-starkware)
workspace_tests/version_integrity_test.rs line 14 at r2 (raw file):
const PARENT_BRANCH: &str = include_str!("../scripts/parent_branch.txt"); const MAIN_PARENT_BRANCH: &str = "main";
Unrelated
Suggestion:
MAIN_BRANCH
dorimedini-starkware
left a comment
There was a problem hiding this comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @amosStarkware, @nimrod-starkware, and @TzahiTaub)
workspace_tests/version_integrity_test.rs line 14 at r2 (raw file):
Previously, TzahiTaub (Tzahi) wrote…
Unrelated
I think it's good that it matches the name of the parent_branch.txt file; how about PARENT_BRANCH? without the MAIN_?
TzahiTaub
left a comment
There was a problem hiding this comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @amosStarkware, @dorimedini-starkware, and @nimrod-starkware)
workspace_tests/version_integrity_test.rs line 14 at r2 (raw file):
Previously, dorimedini-starkware wrote…
I think it's good that it matches the name of the
parent_branch.txtfile; how aboutPARENT_BRANCH? without theMAIN_?
I think you've looked at the wrong const 🙂 . I suggest keeping the PARENT_BRANCH above, and changing the MAIN_PARENT_BRANCH that is a line below, and was probably copied from the first.
dorimedini-starkware
left a comment
There was a problem hiding this comment.
Reviewable status: 1 of 2 files reviewed, all discussions resolved (waiting on @amosStarkware, @nimrod-starkware, and @TzahiTaub)
workspace_tests/version_integrity_test.rs line 14 at r2 (raw file):
Previously, TzahiTaub (Tzahi) wrote…
I think you've looked at the wrong const 🙂 . I suggest keeping the
PARENT_BRANCHabove, and changing theMAIN_PARENT_BRANCHthat is a line below, and was probably copied from the first.
ahhh right you are
done
TzahiTaub
left a comment
There was a problem hiding this comment.
Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @amosStarkware and @nimrod-starkware)
dorimedini-starkware
left a comment
There was a problem hiding this comment.
Reviewed 1 of 1 files at r4, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @amosStarkware and @nimrod-starkware)

No description provided.