fix: remove container after generating protoc

[eryugey]

Add '--rm' option to docker command, so container will be removed
automatically. Otherwise we'll have many exited but not removed
containers as we run 'make protoc'.

Signed-off-by: Eryu Guan [email protected]

Description

Related Issue

Motivation and Context

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation Update (if none of the other choices apply)

Checklist

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.

[codecov-commenter]

Codecov Report

Merging #1306 (1772f12) into main (845a6e1) will decrease coverage by 0.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1306      +/-   ##
==========================================
- Coverage   46.48%   46.12%   -0.36%     
==========================================
  Files         139      139              
  Lines       11716    11716              
==========================================
- Hits         5446     5404      -42     
- Misses       5680     5721      +41     
- Partials      590      591       +1     

Flag	Coverage Δ
Object-compatibility-e2etests	?
e2etests	?
unittests	46.12% <ø> (-0.36%)	⬇️

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

Impacted Files	Coverage Δ
client/daemon/peer/peertask_piecetask_poller.go	30.43% <0.00%> (-16.67%)	⬇️
cdn/supervisor/progress/progress.go	90.62% <0.00%> (-6.25%)	⬇️
...ent/daemon/peer/peertask_piecetask_synchronizer.go	42.56% <0.00%> (-2.57%)	⬇️
client/daemon/peer/peertask_conductor.go	57.84% <0.00%> (-0.73%)	⬇️
scheduler/service/service.go	82.77% <0.00%> (-0.64%)	⬇️

[jim3ma]

LGTM

[anjia0532]

LGTM