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

Feat/opg scanning poc #6

Merged
merged 45 commits into from
Nov 15, 2024
Merged

Feat/opg scanning poc #6

merged 45 commits into from
Nov 15, 2024

Conversation

j1mb0b
Copy link
Contributor

@j1mb0b j1mb0b commented Oct 30, 2024

Purpose

Initial POC setup in order to prepare the skeleton code for the migration from DDC PHP module to Go

Approach

Based on the continued analysis of DDC module we are encapsulating relative handlers for the workflow processing of documents. For the time being the code will represent a high-level outline of how the process is foreseen in Go and is subject to change.

Learning

Good dig of DDC module source code, and the setup of the docker environment (on-going)

Checklist

  • [y ] I have performed a self-review of my own code
  • [y] I have added relevant logging with appropriate levels to my code
  • [y] I have updated documentation where relevant
  • [y ] I have added tests to prove my work
  • [n/a] I have run an accessibility tool against the changes and applied fixes
  • [n/a ] The team have tested these changes

@j1mb0b j1mb0b requested a review from a team as a code owner October 30, 2024 14:46
ramchale
ramchale previously approved these changes Oct 31, 2024
James Mackenzie and others added 2 commits November 4, 2024 11:58
- Configure CodeQL for Go
- Add Dockerfile
- Run tests in pipeline
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

gregtyler and others added 6 commits November 4, 2024 12:29
… queue and update Dockerfile to ensure xsd docs are present.
Log in to ECR, then pull Trivy repositories from it. Use appropriate name for application ECR repository.
service-app/cmd/service/README.md Outdated Show resolved Hide resolved
service-app/.gitignore Outdated Show resolved Hide resolved
service-app/config/config.yml Outdated Show resolved Hide resolved
service-app/docker-compose.yml Outdated Show resolved Hide resolved
service-app/internal/logger/logger.go Outdated Show resolved Hide resolved
service-app/internal/factory/santizer_factory.go Outdated Show resolved Hide resolved
service-app/internal/util/dateparser.go Outdated Show resolved Hide resolved
service-app/internal/factory/document_processor.go Outdated Show resolved Hide resolved
James Mackenzie added 2 commits November 15, 2024 10:01
@j1mb0b j1mb0b requested a review from gregtyler November 15, 2024 10:17
gregtyler
gregtyler previously approved these changes Nov 15, 2024
service-app/internal/types/client.go Outdated Show resolved Hide resolved
service-app/internal/api/client.go Outdated Show resolved Hide resolved
service-app/internal/api/client.go Outdated Show resolved Hide resolved
@gregtyler gregtyler force-pushed the feat/opg-scanning-poc branch from d404faa to 6645984 Compare November 15, 2024 11:57
This service doesn't run its own terraform
So the tests can be run without needing environment variables to be set
@gregtyler gregtyler force-pushed the feat/opg-scanning-poc branch from 19e9864 to 151f688 Compare November 15, 2024 12:33
@j1mb0b j1mb0b merged commit d359f0c into main Nov 15, 2024
14 checks passed
@j1mb0b j1mb0b deleted the feat/opg-scanning-poc branch November 15, 2024 12:55
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.

3 participants