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

Github Actions Test #100

Merged
merged 8 commits into from
Oct 5, 2023
Merged

Github Actions Test #100

merged 8 commits into from
Oct 5, 2023

Conversation

guvenc
Copy link
Collaborator

@guvenc guvenc commented Sep 28, 2023

No description provided.

@github-actions github-actions bot added size/M enhancement New feature or request size/L and removed size/M labels Sep 28, 2023
@damyan damyan force-pushed the feature/test_github_actions branch 2 times, most recently from c209721 to 2c87999 Compare October 5, 2023 11:35
guvenc and others added 8 commits October 5, 2023 16:49
Signed-off-by: Guvenc Gulce <[email protected]>
- remove redundant `check` target
- run `go fmt` from the `check` target
- do not check for license header when running `make test`, use the
  `check` target instead
- do not add license header when building, use the `check` target
  instead
`setup-git-redirect.sh` is needed for tests, because go needs SSH
authentication, when downloading modules dependencies. The redirect
script, though, does not revert the global changes done to the git
config (in that case: SSH instead of HTTPS). The `checkout` action,
on the other hand, needs the HTTPS for checkout, because the SSH key at the
beginning of the action is no longer available, it was properly cleaned
up in the `webfactory/ssh-agent` post clean up.

Implementing workaround as in webfactory/ssh-agent#184
Delete the global git config after lint and test, so we always start the
checkout without URL redirection. See previous commit for details
- kill metalbond
- stop all docker containers
@guvenc guvenc force-pushed the feature/test_github_actions branch from 2c87999 to 8dae25e Compare October 5, 2023 14:52
@guvenc guvenc merged commit 3ca4b8c into main Oct 5, 2023
6 of 7 checks passed
@guvenc guvenc deleted the feature/test_github_actions branch October 5, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants