Skip to content

Conversation

@tortmayr
Copy link
Contributor

What it does

Update maven build to temporarily remove the .gitignore file for diagram resourced during build. This ensures that tycho properly includes the resources in the workflow editor plugin for the p2 update site.

Also fix broken codeql workflow

Fixes eclipse-glsp/glsp/issues/1067

How to test

Inspect nightly p2 site after merge. It should now include the diagram resources

Follow-ups

Changelog

  • This PR should be mentioned in the changelog
  • This PR introduces a breaking change (if yes, provide more details below for the changelog and the migration guide)

Update maven build to temporarily remove the .gitignore file for diagram
resourced during build. This ensures that tycho properly includes the
resources in the workflow editor plugin for the p2 update site.

Also fix broken codeql workflow

Fixes eclipse-glsp/glsp/issues/1067
Copy link
Contributor

@martin-fleck-at martin-fleck-at left a comment

Choose a reason for hiding this comment

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

LGTM!

@tortmayr tortmayr merged commit fcdccd7 into master Dec 11, 2025
5 checks passed
@tortmayr tortmayr deleted the isses/1067 branch December 11, 2025 13:37
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