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

Implement pde.project description interfaces using records #1345

Merged

Conversation

HannesWell
Copy link
Member

The implementations were already immutable and using records avoids implementing some boilerplate code.

And remove duplicated java-doc in BundleProjectService identically copied from the implemented interface.

This is the internal part of #1341.

Copy link

github-actions bot commented Jul 21, 2024

Test Results

   291 files  ±0     291 suites  ±0   54m 12s ⏱️ -8s
 3 580 tests ±0   3 502 ✅  - 2   76 💤 ±0  0 ❌ ±0  2 🔥 +2 
11 037 runs  ±0  10 804 ✅  - 2  231 💤 ±0  0 ❌ ±0  2 🔥 +2 

For more details on these errors, see this check.

Results for commit 5fbd13e. ± Comparison against base commit 2ea2b29.

♻️ This comment has been updated with latest results.

The implementations were already immutable and using records avoids
implementing some boilerplate code.

And remove duplicated java-doc in BundleProjectService identically
copied from the implemented interface.
@HannesWell HannesWell force-pushed the records-for-project-descriptions branch from 0281658 to 5fbd13e Compare July 21, 2024 18:03
@HannesWell HannesWell merged commit 86733d3 into eclipse-pde:master Jul 21, 2024
16 of 17 checks passed
@HannesWell HannesWell deleted the records-for-project-descriptions branch July 21, 2024 20:11
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