Skip to content

ci: use current LiT itest target#1099

Merged
starius merged 1 commit intolightninglabs:masterfrom
starius:fix-lit-test
Mar 18, 2026
Merged

ci: use current LiT itest target#1099
starius merged 1 commit intolightninglabs:masterfrom
starius:fix-lit-test

Conversation

@starius
Copy link
Copy Markdown
Collaborator

@starius starius commented Mar 17, 2026

CI failed: https://github.com/lightninglabs/loop/actions/runs/23082982774/job/67447021668?pr=1090

The workflow called make itest-no-backward-compat, but lightning-terminal removed that target in
lightninglabs/lightning-terminal@4cedb44 ("itest: remove custom channels tests and backward compat infrastructure").

Pull Request Checklist

  • Update release_notes.md if your PR contains major features, breaking changes or bugfixes

CI failed: https://github.com/lightninglabs/loop/actions/runs/23082982774/job/67447021668?pr=1090

The workflow called `make itest-no-backward-compat`, but
lightning-terminal removed that target in
lightninglabs/lightning-terminal@4cedb44
("itest: remove custom channels tests and backward compat infrastructure").
@starius starius requested a review from ellemouton March 17, 2026 21:43
@gemini-code-assist
Copy link
Copy Markdown

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@starius starius marked this pull request as ready for review March 17, 2026 21:58
Copy link
Copy Markdown

@darioAnongba darioAnongba left a comment

Choose a reason for hiding this comment

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

Approved, even though testing this way is an anti-pattern as it's an inversion of responsibilities.

@hieblmi hieblmi self-requested a review March 18, 2026 16:24
Copy link
Copy Markdown
Collaborator

@hieblmi hieblmi left a comment

Choose a reason for hiding this comment

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

LGTM!

@starius starius merged commit 5b3fb0a into lightninglabs:master Mar 18, 2026
9 checks passed
@starius starius deleted the fix-lit-test branch March 18, 2026 16:25
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