#52 - add CI pipeline for pull requests (prettier, tsc, lint, unit tests, sonar) #53

Merged

GuillaumeHemmen merged 2 commits from 52-add-CI-pipeline-for-pull-requests into master 2026-04-10 11:26:13 +00:00

Conversation 0 Commits 2 Files changed 2 +94 -4

GuillaumeHemmen commented 2026-04-10 11:15:36 +00:00

Owner

Copy link

Description

Add a Forgejo Actions CI workflow (.forgejo/workflows/pr.yaml) that runs automatically on every pull request targeting master. The pipeline enforces code quality gates before merging by running Prettier formatting checks, TypeScript type-checking, ESLint linting, Jest unit tests, and SonarQube PR analysis in parallel (except Sonar, which depends on the unit-testing job to consume the coverage artifact).

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Performance improvement
• Refactoring (no functional changes)
• CI/CD update

Related Issue

Fixes #52

Testing

• Unit tests added/updated
• Manual testing performed
• Tested on iOS
• Tested on Android

N/A — this PR adds a CI workflow file only. Validation will be confirmed when the pipeline triggers on the first PR after merge.

Breaking Changes

None.

Checklist

• Code follows the project's code style (Prettier, ESLint)
• TypeScript types are properly defined
• Tests pass locally
• Documentation updated (if needed)
• CI checks pass (Prettier, TypeScript, Lint, Tests, Sonar)

Additional Notes

• All quality jobs (prettier, tsc, lint, unit-testing) run in parallel on the git.van-hemmen.com/actions/sindri:ci-26.8.1 Docker image.
• The sonar job runs after unit-testing and downloads the Jest artifact (test-report.xml + coverage/lcov.info) to feed into a PR-scoped SonarQube analysis using sonar.pullrequest.* parameters.
• Companion workflows for dev and master branches already exist; this PR closes the gap for pull requests.

## Description Add a Forgejo Actions CI workflow (`.forgejo/workflows/pr.yaml`) that runs automatically on every pull request targeting `master`. The pipeline enforces code quality gates before merging by running Prettier formatting checks, TypeScript type-checking, ESLint linting, Jest unit tests, and SonarQube PR analysis in parallel (except Sonar, which depends on the unit-testing job to consume the coverage artifact). ## Type of Change - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] Documentation update - [ ] Performance improvement - [ ] Refactoring (no functional changes) - [x] CI/CD update ## Related Issue Fixes #52 ## Testing - [ ] Unit tests added/updated - [ ] Manual testing performed - [ ] Tested on iOS - [ ] Tested on Android N/A — this PR adds a CI workflow file only. Validation will be confirmed when the pipeline triggers on the first PR after merge. ## Breaking Changes None. ## Checklist - [x] Code follows the project's code style (Prettier, ESLint) - [x] TypeScript types are properly defined - [x] Tests pass locally - [x] Documentation updated (if needed) - [x] CI checks pass (Prettier, TypeScript, Lint, Tests, Sonar) ## Additional Notes - All quality jobs (`prettier`, `tsc`, `lint`, `unit-testing`) run in parallel on the `git.van-hemmen.com/actions/sindri:ci-26.8.1` Docker image. - The `sonar` job runs after `unit-testing` and downloads the Jest artifact (`test-report.xml` + `coverage/lcov.info`) to feed into a PR-scoped SonarQube analysis using `sonar.pullrequest.*` parameters. - Companion workflows for `dev` and `master` branches already exist; this PR closes the gap for pull requests.

GuillaumeHemmen added 1 commit 2026-04-10 11:15:36 +00:00

#52 - Remove redundant needs: install references from dev workflow d54e7b047f

GuillaumeHemmen changed title from #52 - Remove redundant needs: install references from dev workflow to #52 - add CI pipeline for pull requests (prettier, tsc, lint, unit tests, sonar) 2026-04-10 11:15:58 +00:00

GuillaumeHemmen added 1 commit 2026-04-10 11:19:45 +00:00

#52 - Add CI workflow for pull requests with Prettier, TypeScript checks, linting, unit tests, and Sonar analysis 4272aac551

GuillaumeHemmen added the

enhancement

label 2026-04-10 11:25:36 +00:00

GuillaumeHemmen self-assigned this 2026-04-10 11:25:39 +00:00

GuillaumeHemmen merged commit fa74cb0bb3 into master 2026-04-10 11:26:13 +00:00

GuillaumeHemmen deleted branch 52-add-CI-pipeline-for-pull-requests 2026-04-10 11:26:13 +00:00

GuillaumeHemmen referenced this pull request from a commit 2026-04-10 11:26:14 +00:00

#52 - add CI pipeline for pull requests (prettier, tsc, lint, unit tests, sonar) (#53)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

bug

Something isn't working

  

dependencies

Pull requests that update a dependency file

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

question

Further information is requested

  

wontfix

This will not be worked on

No labels

bug

dependencies

documentation

duplicate

enhancement

good first issue

help wanted

invalid

question

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

GuillaumeHemmen

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

openrn/react-native-logging-tools!53

No description provided.