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

Comments

Proposal: Makefile and setting version by tag#11

Merged
athul merged 4 commits intomasterfrom
makefile_and_version_by_tag
Feb 14, 2020
Merged

Proposal: Makefile and setting version by tag#11
athul merged 4 commits intomasterfrom
makefile_and_version_by_tag

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