Skip to content

Add a consumer smoke test for the published scip-kotlin-analysis - #1076

Open
christoph-sg wants to merge 1 commit into
mainfrom
kotlin-analysis-consumer-smoke-test
Open

christoph-sg wants to merge 1 commit into
mainfrom
kotlin-analysis-consumer-smoke-test

Conversation

@christoph-sg

@christoph-sg christoph-sg commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

#1073 fixed a published POM that left scip-kotlin-bindings off consumers' compile classpath, even though KotlinAnalysisIndexer's constructor and run() expose Document. Nothing in CI could catch that, because scip-java, the only in-repo consumer, gets the bindings through scip-aggregator anyway. This adds a small standalone Gradle project under examples/kotlin-analysis-consumer that depends only on the published scip-kotlin-analysis (from mavenLocal(), everything else from Maven Central only). It compiles against the public API and indexes one file. The scip-kotlin-analysis CI job now publishes locally and runs it. Without #1073 it fails with Cannot access class 'org.scip_code.scip.Document'.

Compiles and runs a standalone project that depends only on the
published artifact, so missing api dependencies and references to
artifacts outside Maven Central fail CI.

Amp-Thread-ID: https://ampcode.com/threads/T-01a1ca08-3e68-7113-8182-d86a4ef63ea7
Co-authored-by: Amp <amp@ampcode.com>
@christoph-sg
christoph-sg force-pushed the kotlin-analysis-consumer-smoke-test branch from fff07ee to dbdca48 Compare November 11, 2026 11:30

@kalanchan kalanchan left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

i trust 🙏

This branch has not been deployed

No deployments
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.

2 participants