Skip to content

Add CI workflow to lint and build on PRs#205

Merged
olliewalsh merged 1 commit into
containers:mainfrom
olliewalsh:dep_ci
Jul 17, 2026
Merged

Add CI workflow to lint and build on PRs#205
olliewalsh merged 1 commit into
containers:mainfrom
olliewalsh:dep_ci

Conversation

@olliewalsh

@olliewalsh olliewalsh commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Dependabot bumps were breaking builds but not detected in CI until merged to main and deployed. Add a job to validate PRs.

@olliewalsh
olliewalsh force-pushed the dep_ci branch 2 times, most recently from f8cb18e to 922b9ed Compare July 15, 2026 10:43
@olliewalsh

Copy link
Copy Markdown
Contributor Author

@rhatdan should catch the dependency issues in future

Add playwright smoke test that opens the built site in a headless
browser and verifies the page actually renders. This would have caught
the react/react-dom version mismatch that caused a blank page.

Also fix eslint config to ignore public/ (bundled docs JS) and remove
dead calendar code left behind when the community section was removed.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Oliver Walsh <owalsh@redhat.com>
@olliewalsh
olliewalsh merged commit f049168 into containers:main Jul 17, 2026
2 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.

1 participant