Skip to content

Conversation

@labkey-jeckels
Copy link
Contributor

@labkey-jeckels labkey-jeckels commented Dec 4, 2025

Rationale

Customize build to enable full Java code analysis.

Changes

  • Check out server repo, build together, and push CodeQL results for commonAssays

Changed build mode for Java-Kotlin from 'none' to 'autobuild' to enable Kotlin analysis.
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

ERROR: Branch doesn't match LabKey naming scheme: labkey-jeckels-patch-1
A branch intended for develop should be named something like fb_labkey-jeckels-patch-1

labkey-jeckels and others added 11 commits December 4, 2025 13:51
Changed build mode for Java-Kotlin from autobuild to manual to allow for Kotlin analysis.
Updated CodeQL workflow to checkout multiple LabKey repositories.
Removed exit command after deploying the app.
* Add repository owner reference to CodeQL analysis

Added a reference to the repository owner in CodeQL analysis.

* sha in CodeQL workflow is required if specifying ref
Based on the most recent build error, github.repository_owner wasn't providing the correct data type that we anticipated. Our changes to be more specific about github.sha may have been enough.
For pull requests, `github.ref` points to the read-only merge branch for that PR (e.g. `refs/pull/964/merge`)
Comment out the java-kotlin language configuration and remove checkout step for LabKey/platform.
@labkey-jeckels labkey-jeckels changed the title Enable Kotlin analysis in CodeQL workflow Customize build to enable full Java code analysis Dec 19, 2025
@labkey-jeckels labkey-jeckels merged commit e981488 into develop Dec 19, 2025
5 checks passed
@labkey-jeckels labkey-jeckels deleted the labkey-jeckels-patch-1 branch December 19, 2025 16:47
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.

5 participants