Skip to content

ci: split frontend CI#1196

Merged
lusergit merged 2 commits intoedgehog-device-manager:mainfrom
lusergit:push-okvozyrqutrt
Jan 28, 2026
Merged

ci: split frontend CI#1196
lusergit merged 2 commits intoedgehog-device-manager:mainfrom
lusergit:push-okvozyrqutrt

Conversation

@lusergit
Copy link
Copy Markdown
Collaborator

Split frontend CI

Up to now the frontend CI was running in sequence for different jobs. This PR splits different jobs into actual different jobs to get the following advantages:

  • It's clearer what failed and what passes: linting, translations and type checks are separate: let's reflect that in CI
  • It's faster: even if cache warmup takes a bit of overhead, that's not that problematic

On a side not badges were added to the README to make it a little bit cuter :3

Checklist

  • I have read the CONTRIBUTING.md
  • I have added tests that prove my fix is effective or that my feature works
  • I have added or updated documentation (if appropriate)

@lusergit lusergit force-pushed the push-okvozyrqutrt branch 2 times, most recently from 5b5b493 to aaefeef Compare January 27, 2026 10:49
@github-advanced-security
Copy link
Copy Markdown

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.

@lusergit lusergit force-pushed the push-okvozyrqutrt branch 3 times, most recently from aaf949a to cd488c1 Compare January 27, 2026 11:04
@coveralls
Copy link
Copy Markdown

coveralls commented Jan 27, 2026

Pull Request Test Coverage Report for Build 21395091740

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-77.8%) to 0.0%

Totals Coverage Status
Change from base Build bceff82cb5b7807bdb284c04a9a210c986a91c26: -77.8%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

@lusergit lusergit force-pushed the push-okvozyrqutrt branch 2 times, most recently from a320cc6 to 53e6fe5 Compare January 27, 2026 11:15
Splits frontend CI to have different jobs. Each job takes care of one aspect of
the CI and if it fails it's clear what failed and other steps that might have
failed to.

Signed-off-by: Luca Zaninotto <luca.zaninotto@secomind.com>
Adds badges to make the readme a bit nicer :3

Signed-off-by: Luca Zaninotto <luca.zaninotto@secomind.com>
@lusergit
Copy link
Copy Markdown
Collaborator Author

/fast-forward

@lusergit lusergit merged commit 4ccb2cd into edgehog-device-manager:main Jan 28, 2026
16 checks passed
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.

5 participants