Skip to content

Conversation

@TennyZhuang
Copy link
Contributor

Signed-off-by: TennyZhuang [email protected]

What's changed and what's your intention?

PLEASE DO NOT LEAVE THIS EMPTY !!!

prototool is not maintained, switch to buf.

Checklist

  • I have written necessary docs and comments
  • I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Close #2008

Signed-off-by: TennyZhuang <[email protected]>
@TennyZhuang TennyZhuang requested a review from neverchanje April 24, 2022 09:29
@codecov
Copy link

codecov bot commented Apr 24, 2022

Codecov Report

Merging #2088 (169b87f) into main (d57c97d) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2088      +/-   ##
==========================================
- Coverage   71.05%   71.05%   -0.01%     
==========================================
  Files         635      635              
  Lines       81620    81620              
==========================================
- Hits        57999    57997       -2     
- Misses      23621    23623       +2     
Flag Coverage Δ
rust 71.05% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/connector/src/filesystem/file_common.rs 80.00% <0.00%> (-0.89%) ⬇️

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

@TennyZhuang TennyZhuang merged commit eacbd75 into main Apr 24, 2022
@TennyZhuang TennyZhuang deleted the ci/retire-prototool branch April 24, 2022 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: migrate from uber/prototool to bufbuild/buf

3 participants