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

Add Balta Test into CI & plpgsql_check into the repo - ideally into CICD pipelines #155

Open
lsulak opened this issue Feb 8, 2024 · 0 comments
Labels
DB Issues touching the Database part of the project enhancement New feature or request tests QA/test-specific

Comments

@lsulak
Copy link
Collaborator

lsulak commented Feb 8, 2024

Background

As we are increasingly using SQL (and pl/pgSQL), it might be a good idea to be more strict / have enforced or semi-encforced standards about our SQL code. This tool seems to be covering a lot of such needs: https://github.com/okbob/plpgsql_check/.

Also, add a CI check for the Balta tests so that it's all automatic and mandatory.

Currently we are testing our database functions with Balta: https://github.com/AbsaOSS/balta but we don't know what is our code coverage. Also, it might be good to perform some sort of static check code analysis (linting etc) on our SQL code.

The goal of this ticket is to include this tool into the project so that it's convenient and brings value for us.

@lsulak lsulak added the enhancement New feature or request label Feb 8, 2024
@lsulak lsulak added the DB Issues touching the Database part of the project label Feb 8, 2024
@benedeki benedeki added the tests QA/test-specific label Feb 29, 2024
@lsulak lsulak changed the title Add plpgsql_check into the repo - ideally into CICD pipelines Add Balta Test into CI & plpgsql_check into the repo - ideally into CICD pipelines Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DB Issues touching the Database part of the project enhancement New feature or request tests QA/test-specific
Projects
Status: 🆕 To groom
Development

No branches or pull requests

2 participants