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

Update Github Action workflows #2325

Merged
merged 10 commits into from
Oct 17, 2022
Merged

Update Github Action workflows #2325

merged 10 commits into from
Oct 17, 2022

Conversation

DavisRayM
Copy link
Contributor

@DavisRayM DavisRayM commented Oct 13, 2022

Changes / Features implemented

Static Analysis -> Unit Tests
&
Trivy security checks

Steps taken to verify this change does what is intended

  • Manually built image

Side effects of implementing this change

N/A

@DavisRayM DavisRayM force-pushed the include-postgres-client branch 13 times, most recently from 6c86411 to 730c46e Compare October 13, 2022 13:19
- Include security scan job that runs in parallel with the CI and Static
  analysis checks
ukanga
ukanga previously approved these changes Oct 13, 2022
@DavisRayM DavisRayM requested a review from ukanga October 13, 2022 14:02
ukanga
ukanga previously approved these changes Oct 13, 2022
@DavisRayM DavisRayM changed the title Add PostgreSQL client to the Docker images Update Github Action workflows Oct 13, 2022
The testcase shares a common folder that needs to be removed after each
test case; In cases where the tests are running in parallel you can get
a scenarion where Test Case A is deleting the folder while Test case B
is adding a new file. Which would case an OS Error: Directory not empty
to be raised
ukanga
ukanga previously approved these changes Oct 13, 2022
- Silence codacy docstring error
@DavisRayM DavisRayM merged commit 1af1dd1 into main Oct 17, 2022
@DavisRayM DavisRayM deleted the include-postgres-client branch October 17, 2022 08:05
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.

2 participants