Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove unnecessary dependencies from Makefile targets #56

Merged
merged 1 commit into from
Dec 4, 2023
Merged

Conversation

tagatac
Copy link
Owner

@tagatac tagatac commented Dec 4, 2023

What is changing:

  • Stop including tools/cover as a dependency of bagoup.
  • Stop running clean and deps targets before generate.
  • Use long versions of commandline flags where possible.

Why this change is being made:

  • Get rid of some noise when running these targets.
  • Make the commandline flags more readable.

Related issue(s): N/A

Follow-up changes needed: None

Is the change completely covered by unit tests? If not, why not?: N/A

@tagatac tagatac merged commit 689cb7f into main Dec 4, 2023
@tagatac tagatac deleted the deps branch December 4, 2023 15:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant