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

Initialize the main branch #1172

Merged
merged 15 commits into from
Aug 5, 2022
Merged

Initialize the main branch #1172

merged 15 commits into from
Aug 5, 2022

Conversation

thanethomson
Copy link
Contributor

Partially addresses #1168.

This is a large PR because it involves reformatting all of the code on the v0.23.x branch, which will form the basis for the new main branch.

thanethomson and others added 12 commits August 2, 2022 12:35
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
* Pin `time` to <0.3.12 to ensure we stay no-std compatible

* Apply suggestions from code review

* Update proto/Cargo.toml

Co-authored-by: Thane Thomson <thane@informal.systems>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@codecov-commenter
Copy link

codecov-commenter commented Aug 4, 2022

Codecov Report

Merging #1172 (d35e7e6) into main (590a038) will increase coverage by 0.0%.
The diff coverage is 93.2%.

@@          Coverage Diff          @@
##            main   #1172   +/-   ##
=====================================
  Coverage   67.5%   67.5%           
=====================================
  Files        214     214           
  Lines      20474   20480    +6     
=====================================
+ Hits       13828   13835    +7     
+ Misses      6646    6645    -1     
Impacted Files Coverage Δ
abci/src/application.rs 18.0% <ø> (ø)
abci/src/application/kvstore.rs 74.4% <ø> (ø)
abci/src/client.rs 40.0% <ø> (ø)
abci/src/codec.rs 90.1% <ø> (ø)
abci/src/lib.rs 100.0% <ø> (ø)
abci/src/server.rs 9.4% <ø> (-0.1%) ⬇️
abci/tests/kvstore_app.rs 100.0% <ø> (ø)
config/src/config.rs 55.0% <0.0%> (ø)
config/src/error.rs 0.0% <ø> (ø)
config/src/net.rs 92.2% <ø> (ø)
... and 192 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

thanethomson and others added 3 commits August 5, 2022 07:55
* Bump Prost version to v0.11

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add changelog entry

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add step in generated-protos-compile job to install protoc

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* shell should be run

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Tell curl to follow redirects

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Fix protoc unzip target

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Install protoc to unprivileged location

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Regenerate protos

Signed-off-by: Thane Thomson <connect@thanethomson.com>
* Bump version to v0.23.9

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Prepare changelog release

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Rebuild changelog

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add note on time dependency restriction to changelog

Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@thanethomson thanethomson marked this pull request as ready for review August 5, 2022 18:02
@thanethomson thanethomson merged commit dda9245 into main Aug 5, 2022
@thanethomson thanethomson deleted the thane/init-main branch August 5, 2022 18:21
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