Skip to content

Add Volto Light Theme frontend CI workflow - #8516

Open
wesleybl wants to merge 1 commit into
mainfrom
ci-vlt
Open

wesleybl wants to merge 1 commit into
mainfrom
ci-vlt

Conversation

@wesleybl

@wesleybl wesleybl commented Nov 7, 2026

Copy link
Copy Markdown
Member

Run the volto-light-theme frontend test suite on push and pull request. The workflow checks out the main branch of kitconcept/volto-light-theme and replaces its core folder with the Volto branch that triggered the workflow before installing the dependencies.

It then runs lint, Storybook build, i18n checks, and the unit tests. The install uses --no-frozen-lockfile so dependency changes from the Volto branch are resolved correctly.

Run the volto-light-theme frontend test suite on push and pull request.
The workflow checks out the main branch of kitconcept/volto-light-theme
and replaces its core folder with the Volto branch that triggered the
workflow before installing the dependencies.

It then runs lint, Storybook build, i18n checks, and the unit tests.
The install uses --no-frozen-lockfile so dependency changes from the
Volto branch are resolved correctly.
@wesleybl

wesleybl commented Nov 7, 2026

Copy link
Copy Markdown
Member Author

@sneridagh It is necessary to remove the Node 22 jobs from the CI configuration.

@wesleybl

wesleybl commented Nov 7, 2026

Copy link
Copy Markdown
Member Author

@sneridagh I don’t think a failure in this job should block the merge. A failure in VLT tests must not prevent the merging of a Volto PR.

@davisagli davisagli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a good idea. I think it would be a lot more meaningful to run VLT's acceptance tests rather than just the unit tests, but I'm not sure how long the build would take.

@davisagli

Copy link
Copy Markdown
Member

@wesleybl @sneridagh I updated the branch protection rules for the main branch to require the Node 26 jobs instead of the Node 22 ones

@sneridagh

Copy link
Copy Markdown
Member

@davisagli thanks, I was going to do it several times but got sidetracked.

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.

4 participants