Skip to content

Conversation

@jalseth
Copy link
Member

@jalseth jalseth commented Oct 4, 2025

No description provided.

@jalseth
Copy link
Member Author

jalseth commented Oct 4, 2025

This is currently blocked on #1179 and #1180.

@jalseth jalseth requested review from boranx and jpreese October 5, 2025 18:20
@jpreese
Copy link
Member

jpreese commented Oct 6, 2025

Change makes sense to me, we'll just want to validate the pipelines after merging in the other PRs

@jalseth jalseth force-pushed the jalseth/pr-ci-all-os branch 3 times, most recently from fc16aea to 56dbc29 Compare October 7, 2025 23:30
Previously the plugin tests were failing on Windows due to the tests
being written only for *nix systems.

Signed-off-by: James Alseth <[email protected]>
This test depends on *nix permissions which was broken on Windows.

Signed-off-by: James Alseth <[email protected]>
The bats-core/bats-action does not set $PATH correctly on Windows.

Signed-off-by: James Alseth <[email protected]>
@jalseth jalseth force-pushed the jalseth/pr-ci-all-os branch 3 times, most recently from 599b572 to 10ab8ab Compare November 9, 2025 01:48
Many of these are not passing due to OS-specific behavior.

Signed-off-by: James Alseth <[email protected]>
@jalseth jalseth force-pushed the jalseth/pr-ci-all-os branch from c7eaa77 to 884c07e Compare November 9, 2025 01:59
@jalseth
Copy link
Member Author

jalseth commented Nov 9, 2025

This required some fixes, and disabling some tests that appear to unintentionally be Linux-specific. We can follow up on those in the future.

@jalseth jalseth merged commit 7d41238 into master Nov 9, 2025
14 checks passed
@jalseth jalseth deleted the jalseth/pr-ci-all-os branch November 9, 2025 02:09
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