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

[TEST-93] Updated Makefile #58

Merged
merged 2 commits into from
Jun 30, 2022
Merged

[TEST-93] Updated Makefile #58

merged 2 commits into from
Jun 30, 2022

Conversation

sampocs
Copy link
Collaborator

@sampocs sampocs commented Jun 28, 2022

Summary

  • Removed anything from the Makefile that wasn't used (we can always add it back later if we need to)
  • Removed local install steps as it was not necessary to download packages and it would cause changes to the submodules that would be displayed in when running git status
  • Added clean steps
  • Changed names of local dev commands

Test plan

  • Execute each make command (the test commands will fail until we fix the unit tests)

@sampocs sampocs changed the title [TEst-93] Updated Makefile [TEST-93] Updated Makefile Jun 28, 2022
@shellvish
Copy link
Contributor

Looks great to me!

@sampocs sampocs merged commit ed893a5 into main Jun 30, 2022
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.

2 participants