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

Initial repo setup #9

Merged
merged 5 commits into from
Sep 10, 2020
Merged

Initial repo setup #9

merged 5 commits into from
Sep 10, 2020

Conversation

MakMuftic
Copy link
Member

@MakMuftic MakMuftic commented Sep 10, 2020

  • add codeowners
  • add test pipeline
  • add command example
  • write basic makefile

closes #1

Mak Muftic added 2 commits September 10, 2020 13:21
Add github setup, initial packages and sample command, reading config file and env variables, create Makefile.
@MakMuftic MakMuftic self-assigned this Sep 10, 2020
@MakMuftic MakMuftic requested a review from mpetrun5 September 10, 2020 12:39
@MakMuftic MakMuftic merged commit fbb40b5 into master Sep 10, 2020
@MakMuftic MakMuftic deleted the mmuftic/repo-setup branch September 10, 2020 12:48
Copy link
Member

@mpetrunic mpetrunic left a comment

Choose a reason for hiding this comment

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

@MakMuftic @mpetrun5 I've added branch protection so this hasty merges cannot happen anymore. Pls apply same stuff to vedran-daemon before anything else

go-version: 1.13.x

- name: Checkout code
uses: actions/checkout@v2
Copy link
Member

Choose a reason for hiding this comment

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


## Installation
You can download already built binaries for your platform from `builds` folder or get `vedran` golang package and build it locally. Find detailed instructions below.
Copy link
Member

Choose a reason for hiding this comment

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

We shouldn't keep builds in git but we would have them on github release page artifacts

```

## Development
Copy link
Member

Choose a reason for hiding this comment

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

add instructions for clone, install dependencies, run, test

Run daemon app with `go run main.go [command]`.

Expected name of the configuration file depends on `ENV` variable. For example, if you run a daemon app with `ENV=test go run main.go start`, expected config file name is `config-test.yaml`
Copy link
Member

Choose a reason for hiding this comment

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

Why do we need config-test.yaml?
Everything should be configured via cli flags

jobs:
build:
name: Run tests
runs-on: ubuntu-latest
Copy link
Member

Choose a reason for hiding this comment

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

should be ran on all 3 os-es. We can reduce build time to only run on all 3 os-es on PR and only ubuntu on push but since it's oss, it's free anyways

@MakMuftic MakMuftic mentioned this pull request Sep 11, 2020
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.

Setup go repository for Load Balance
3 participants