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

chore(crit): improve Makefile rule #141

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

snprajwal
Copy link
Member

Previously, the Makefile in crit would not rebuild the binary even if any of the related source files changed. This is now fixed by marking all Go files under the directory as prerequisites for the bin/crit recipe.

Previously, the Makefile in `crit` would not rebuild the binary even if
any of the related source files changed. This is now fixed by marking
all Go files under the directory as prerequisites for the `bin/crit`
recipe.

Signed-off-by: Prajwal S N <prajwalnadig21@gmail.com>
@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (613fd74) 50.34% compared to head (17e1120) 50.34%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #141   +/-   ##
=======================================
  Coverage   50.34%   50.34%           
=======================================
  Files          24       24           
  Lines        2590     2590           
=======================================
  Hits         1304     1304           
  Misses       1102     1102           
  Partials      184      184           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@snprajwal snprajwal merged commit d52dc0d into checkpoint-restore:master Aug 1, 2023
14 checks passed
@snprajwal snprajwal deleted the improve-make branch August 1, 2023 15:39
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.

None yet

3 participants