Skip to content
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

[UIMA-6462] Avoid deploy broken checksum files for p2content.xml and artifacts.xml in feature modules #205

Conversation

reckart
Copy link
Member

@reckart reckart commented May 23, 2022

JIRA Ticket: https://issues.apache.org/jira/browse/UIMA-6462

What's in the PR

  • Prevent feature modules from being deployed
  • Add a marker file to the project root to trigger the Tycho release extensions

How to test manually

  • Run the deploy goal and check that is has no effect for the feature modules

Automatic testing

  • PR adds/updates unit tests

Documentation

  • PR adds/updates documentation

Organizational

  • PR includes new dependencies.
    Only dependencies under approved licenses are allowed. LICENSE and NOTICE files in the respective modules where dependencies have been added as well as in the project root have been updated.

…artifacts.xml in feature modules

- Prevent feature modules from being deployed
- Add a marker file to the project root to trigger the Tycho release extensions
@reckart reckart added the 🦟 Bug Something isn't working label May 23, 2022
@reckart reckart added this to the 3.4.0 milestone May 23, 2022
@reckart reckart self-assigned this May 23, 2022
@reckart reckart merged commit 11dcc08 into main May 23, 2022
@reckart reckart deleted the bugfix/UIMA-6462-Avoid-deploy-broken-checksum-files-for-p2content.xml-and-artifacts.xml-in-feature-modules branch May 23, 2022 08:22
reckart added a commit that referenced this pull request Sep 23, 2024
- Validate managed (J)CASes by default
reckart added a commit that referenced this pull request Sep 23, 2024
…s-should-consider-validators

Issue #205: Managed CASes in tests should consider validators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦟 Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant