Unify nature checks and reuse existing constants from core.natures.PDE

[HannesWell]

No description provided.

[github-actions]

Test Results

   291 files  ±0     291 suites  ±0   56m 35s ⏱️ + 1m 35s
 3 580 tests ±0   3 504 ✅ ±0   76 💤 ±0  0 ❌ ±0 
11 037 runs  ±0  10 806 ✅ ±0  231 💤 ±0  0 ❌ ±0 

Results for commit 0e589ca. ± Comparison against base commit 8bc60ea.

♻️ This comment has been updated with latest results.

[HannesWell]

I have now taken this as an opportunity to replace all occurrences of PDE nature and builder literals by the corresponding constants in the PDE class and to leveraging the different PDE.hasXNature methods.

Additionally I moved the isBndProject(IProject) method from BndProject to the PDE class to align it with the other methods.

I also contemplate to rename the hasXNature() method to isXProject(), which I would consider a more speaking name.
But if these names are changed, we could alternativly consider to do the opposite and move the hasXNature respectivly isXProject methods as well as the constants to the corresponding XProject classes in the same package, just like it used to be for the BndProject class.

What's the opinion of the others? Keep one central class for all nature and builder ids and hasNature() or isProject() methods? Or should those constants and methods be added to the corresponding project class?
At the moment we have a split between BNDProject and Plugin/Feature/SiteProject and I would like to unify this.

[laeubi]

I would appreciate if we could delay such cleanups/unification until the pending PRs are merged so they don't get delayed/distorted by rebase conflicts, for me central classes tend to be convoluted by more and more unrelated stuff over time so I think keeping it on the project classes seems good.

[laeubi]

@HannesWell still relevant?

[HannesWell]

I also contemplate to rename the hasXNature() method to isXProject(), which I would consider a more speaking name.
But if these names are changed, we could alternativly consider to do the opposite and move the hasXNature respectivly isXProject methods as well as the constants to the corresponding XProject classes in the same package, just like it used to be for the BndProject class.

Implemented that now which makes the PDE class obsolete since all methods and constants have been moved to the corresponding <Type>Project classes. Furthermore I renamed BaseProject to PDEProject since fundamental constants and methods reside there now and I think the latter name more expressive.

Now I just have to find out, why the tests are failing...

[HannesWell]

This should finally work.
@laeubi or @merks do you have any remarks on this one?

[HannesWell]

These quality-gates are really pedantic, but in the end that's good. And now finally the build is green. 🎉
Submitting.
Thanks to everyone that participated.