Skip to content

Fix test-framework resolution and hs-Cpp compilation across platforms#31

Open
meanmail wants to merge 3 commits intomainfrom
fix-ci-and-tests
Open

Fix test-framework resolution and hs-Cpp compilation across platforms#31
meanmail wants to merge 3 commits intomainfrom
fix-ci-and-tests

Conversation

@meanmail
Copy link
Copy Markdown
Contributor

@meanmail meanmail commented Mar 26, 2026

Summary

  • Fix test-framework dependency resolution — remove explicit version parameter so IntelliJ Platform Gradle Plugin resolves it automatically from the configured platform
  • Fix hs-Cpp compilation across all platforms (252, 253, 261) by adding platform-specific cmakeCompat.kt with the correct cmake dependency resolution

Test plan

  • Verify CI passes for all platforms (252, 253, 261)
  • Confirm hs-Cpp module compiles successfully on all platforms
  • Confirm hs-core tests can resolve test-framework and run

🤖 Generated with Claude Code

- Remove explicit version from testFramework() — let the plugin resolve
  it automatically from the configured platform dependency
- Extract makeCmakeExecutable() into platform-specific branches:
  252/253 use getBundledCMakeToolBinary(Boolean, ToolKind) API,
  261 uses getBundledCMakeToolBinary(ToolKind) API

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@meanmail meanmail changed the title CI: split workflows, fix tests and hs-Cpp compilation Fix test-framework resolution and hs-Cpp compilation across platforms Mar 26, 2026
meanmail and others added 2 commits March 26, 2026 18:26
Module tests (hs-core, hs-Java, etc.) cannot run directly because
prepareSandbox is disabled in modules, leaving IntelliJ Platform
jars off the test runtime classpath. Run :intellij-plugin:test
(which includes hs-core tests) plus standalone module tests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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