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

ci(*): add makefile #60

Closed
wants to merge 5 commits into from
Closed

Conversation

vdice
Copy link
Member

@vdice vdice commented Aug 17, 2023

This is a little bit of a smorgasbord of CI additions and am open to scoping to just the ones desired. My primary motivation was to reduce the build->package->install cycle during development, hence the Makefile and associated targets, doing away with the need to manually edit cloud.json and install, etc. I figured while doing so, I would add lint and test targets and use these in CI so that devs and automation are invoking the same commands.

Signed-off-by: Vaughn Dice <vaughn.dice@fermyon.com>
Signed-off-by: Vaughn Dice <vaughn.dice@fermyon.com>
Signed-off-by: Vaughn Dice <vaughn.dice@fermyon.com>
Signed-off-by: Vaughn Dice <vaughn.dice@fermyon.com>
Signed-off-by: Vaughn Dice <vaughn.dice@fermyon.com>
@vdice vdice mentioned this pull request Aug 17, 2023
@vdice vdice marked this pull request as draft August 17, 2023 15:26
@vdice
Copy link
Member Author

vdice commented Aug 17, 2023

Putting back in draft per discussion in #36

@vdice
Copy link
Member Author

vdice commented Aug 25, 2023

With #72 the local dev loop is much improved. I'll close this for now and perhaps we can return with some CI improvements/consolidation in the near future.

@vdice vdice closed this Aug 25, 2023
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.

1 participant