Skip to content

10 GitHub actions setup#19

Merged
thomasdejaeghere merged 14 commits intodevelopmentfrom
10-github-actions-setup
Feb 26, 2025
Merged

10 GitHub actions setup#19
thomasdejaeghere merged 14 commits intodevelopmentfrom
10-github-actions-setup

Conversation

@LambrechtMaarten
Copy link
Copy Markdown
Contributor

Implementation of a CI (continuous integration) using github actions via a self-hosted runner.
The CI will execute the github actions and will lint and test the code upon a pull request/push, on the main/development branch.

Simple tests and configurations were used, but can be changed in the future to further integrate more detailed checks.

Deployment will be added later because right now I want to make sure that the testing and linting actions work correctly, and are able to run on our self-hosted runner (locally you can only test your Github actions on the Github runners).

@bramcomyn bramcomyn added the enhancement New feature or request label Feb 25, 2025
@bramcomyn bramcomyn added this to the Milestone 1 milestone Feb 25, 2025
@bramcomyn bramcomyn linked an issue Feb 25, 2025 that may be closed by this pull request
@bramcomyn
Copy link
Copy Markdown
Contributor

Perhaps it would be wiser to also add testing requirements on each branch we try to merge into another? That way, we only work with code that is guaranteed to work.

@bramcomyn bramcomyn self-requested a review February 25, 2025 18:12
@LambrechtMaarten
Copy link
Copy Markdown
Contributor Author

The actions will now execute on every branch, I also made some changes to the rights on the server that caused the first commit to not be accepted.

@thomasdejaeghere thomasdejaeghere merged commit f7aa4cd into development Feb 26, 2025
2 checks passed
@thomasdejaeghere thomasdejaeghere deleted the 10-github-actions-setup branch February 26, 2025 12:47
@LambrechtMaarten LambrechtMaarten removed a link to an issue Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants