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.

Sign up for GitHub

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

Jump to bottom

fix: ci, test, build and docs #18

Merged
merged 26 commits into from
Oct 19, 2022
Merged

fix: ci, test, build and docs #18

merged 26 commits into from
Oct 19, 2022

Conversation

pratikbin
Copy link
Contributor

@pratikbin pratikbin commented Sep 29, 2022 •
edited
Loading

  • Refactor dockerfile

    • Multi stage build
    • ubuntu as base image
    • Caching support for go modules and go build cache at docker level
    • assetClient as entrypoint

  • Makefile/Build

    • Add -s -w ldflag and -trimpath to downsize the binary, more here
    • Add docker compose support

  • Refactor CI

    • deploy: Remove
    • lint:
      • bump to latest
      • trigger only on go code changes
    • Build CI
      • Remove gopkg
      • Cache go modules at CI level
      • Use go version 14
    • Add Mocha
    • Rename and refactor jmeter tests

  • Merged mocha testing branch sarvagya/testing

    • Remove always zero exit
    • Move to .mocha

pratikbin and others added 26 commits September 27, 2022 14:15
fix(dockerfile): multi stage with ubuntu base img
81c531e
feat(build): add docker compose, furnish makefile
b2f7fca
fix(ci): upgrade actions, furnish ci, remove deploy
51c72aa
fix(docs): furnish readme
332789d
fix(build): interactive docker compose
63f4b44
fix(docs): smaller contributors badge, remove sonarcloud
e5cdf29
@iamsarvagyaa
test: structured dirs for mocha
55feec6
@iamsarvagyaa
test: initiated test w/ mocha
de856a9
@iamsarvagyaa
test: previous tests
5c421ed
@iamsarvagyaa
test: testcases w/ html output
d1d2418
test: jmeter in ci
42a374a
ci: enable manually trigger
eeb47f7
ci: lint on only go code change
e60665b
ci: build image before compose
a019363
fix(ci): image cache from and to
f3cd0ff
ci: upload jmeter report
2c7a6e8
ci: mocha test, keyring back test, mv test yaml
47ac563
ci: rename jobs name
9f6093e
Merge branch 'sarvagya/testing' into pratikbin/ci
19be82d
ci: fix moch install, always clean docker
bd6f715
test: mocha furnish
1a44990
ci: enable non zero exit
f4deafa
@pratikbin
ci: fix networking in docker compose
861a431
@iamsarvagyaa
test: testcases w/ jmeter
5ee330f
@pratikbin
fix(docs): YT links, contrib banner size, indent...
1797933
@pratikbin
Merge branch 'sarvagya/testing' into pratikbin/ci
f6ac6f8
@pratikbin pratikbin changed the title [WIP] fix: ci, test, build and docs fix: ci, test, build and docs Oct 19, 2022
deepanshutr
deepanshutr approved these changes Oct 19, 2022
View reviewed changes
Copy link
Member

@deepanshutr deepanshutr left a comment

Choose a reason for hiding this comment

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

Reviewed

@deepanshutr deepanshutr merged commit 2697255 into deepanshutr/cleanup Oct 19, 2022
@deepanshutr deepanshutr deleted the pratikbin/ci branch October 25, 2022 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deepanshutr deepanshutr deepanshutr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pratikbin @deepanshutr @iamsarvagyaa