Skip to content

Run CI against LavinMQ too#201

Merged
walro merged 3 commits intomainfrom
ci/lavinmq
Apr 13, 2026
Merged

Run CI against LavinMQ too#201
walro merged 3 commits intomainfrom
ci/lavinmq

Conversation

@walro
Copy link
Copy Markdown
Contributor

@walro walro commented Apr 13, 2026

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitHub Actions setup to expand broker coverage by adding a LavinMQ CI workflow, while separating lint/format/typecheck into a dedicated workflow and renaming the existing RabbitMQ workflow for clarity.

Changes:

  • Renamed the RabbitMQ workflow and removed lint/format/typecheck steps from it.
  • Added a new lint.yml workflow to run linting, formatting checks, and typechecking independently.
  • Added a new lavinmq.yml workflow to run the test suite (and browser tests) against LavinMQ.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/rabbitmq.yml Renames the workflow and narrows it to running tests only (lint/typecheck moved out).
.github/workflows/lint.yml New standalone workflow for lint/format/typecheck.
.github/workflows/lavinmq.yml New workflow to run Node + browser tests against LavinMQ, including TLS setup.

Comment thread .github/workflows/lint.yml
Comment thread .github/workflows/lavinmq.yml
Comment thread .github/workflows/lavinmq.yml
@walro walro marked this pull request as ready for review April 13, 2026 12:30
@walro walro merged commit 91623e8 into main Apr 13, 2026
20 of 21 checks passed
@walro walro deleted the ci/lavinmq branch April 13, 2026 12:57
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.

3 participants