Skip to content

Enable Tycho clean-ups for platform #1877

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vogella opened this issue May 16, 2025 · 1 comment · May be fixed by #1878
Open

Enable Tycho clean-ups for platform #1877

vogella opened this issue May 16, 2025 · 1 comment · May be fixed by #1878
Labels
enhancement New feature or request

Comments

@vogella
Copy link
Contributor

vogella commented May 16, 2025

If I run the clean-ups we also use in our Tycho build for platform, I get lots of clean-ups. But I see no PR for these clean-ups created by Tycho.

I suggest we also activate them here. Or are they paused for a reason? How can I check that? In Equinox I see some ignore configuration but PDE does not seem to have a configuration for cleanup.

Equinox:

     <groupId>org.eclipse.tycho</groupId>
          <artifactId>tycho-cleancode-plugin</artifactId>
          <configuration>
              <ignores>
                  <ignore>felix/src/.*</ignore>
                  <ignore>osgi/src/.*</ignore>
                  <ignore>felix/src_test/.*</ignore>
              </ignores>
          </configuration>
      </plugin>

How are these clean-ups activated for a repository?

@vogella vogella added the enhancement New feature or request label May 16, 2025
@laeubi
Copy link
Contributor

laeubi commented May 16, 2025

This is planned for the next release stream, to not create too much rumor now we are in RC phase.

The cleanup is activated by a timed github action for example here:
https://github.com/eclipse-pde/eclipse.pde/blob/master/.github/workflows/doCleanCode.yml

@laeubi laeubi linked a pull request May 16, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants