Skip to content
This repository has been archived by the owner on Jul 14, 2022. It is now read-only.

Proposal: Makefile and setting version by tag #11

Merged
merged 4 commits into from
Feb 14, 2020

Conversation

gbmor
Copy link
Contributor

@gbmor gbmor commented Feb 14, 2020

I added a short Makefile that makes it a bit easier for someone to quickly build/install/update using make. The version is set in cli.go during buildtime to the highest version tag by building this way.

When I added the make instructions to README.md it appears VSCode autoformatted, so apologies for the whitespace. But, I figured it was harmless.

I don't know if this is a change you would want, but I figured why not propose it? If you do want this, feel free to merge it. And if not, feel free to close the PR instead (I won't be offended).

@athul
Copy link
Contributor

athul commented Feb 14, 2020

Woah, that was quite fast. Thanks for this

I'd suggest to update the .travis.yml file too since this would make it easier to install and all. But the CI doesn't do much other than Download the dependencies, build it , run it and rm -rf it 😝. I guess we'll need to update that in the future.

Other than that.... LGTM 👍

Copy link
Contributor

@athul athul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@athul athul merged commit d8b54f6 into master Feb 14, 2020
@athul athul deleted the makefile_and_version_by_tag branch September 3, 2020 05:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants