-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
update documentation for building on darwin #4796
Conversation
Welcome @ivelichkovich! |
/assign @ianlewis |
Codecov Report
@@ Coverage Diff @@
## master #4796 +/- ##
==========================================
- Coverage 45.43% 45.42% -0.02%
==========================================
Files 214 214
Lines 9977 9977
Branches 94 94
==========================================
- Hits 4533 4532 -1
- Misses 5180 5181 +1
Partials 264 264
Continue to review full report at Codecov.
|
Add that before |
Moved it, I can move it around more or change the text if needed |
docs/developer/getting-started.md
Outdated
@@ -82,6 +82,12 @@ Open a browser and access the UI under `localhost:9090`. The following processes | |||
|
|||
In order to package everything into a ready-to-run Docker image, use the following task: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This part should be after export GOOS=linux
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah yeah that makes more sense, updated
while we're at it, is there a faster way to build this docker image? If I'm only changing the go for example, the frontend takes ages to build, is there a way I can just manually run go build and build a docker container from that? |
Also what should I do about the test coverage? |
We do not have separate task for that, but you could manually compile backend binary, replace it in the dist directory and run only
Don't worry about that. Those calculations have a margin of error. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: floreks, ivelichkovich The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
otherwise you can't build docker when working in a darwin environment, you get incompatible architectures from go build.