eclipse-plugins icon indicating copy to clipboard operation
eclipse-plugins copied to clipboard

Cleanup warnings

Open jonahgraham opened this issue 5 years ago • 1 comments

The embedcdt plug-ins are quite close to warning free. I think a little push can get us there, or close enough to make it easish to tell there are warnings.

There are four cases of warnings I see that cover most of the warnings

  • lots of warnings in liqp - If this was moved into a new bundle, the warnings for this bundle could be set to info. This requires .settings to be checked in though.
  • lots of provisional API usage - This can be solved in two ways - one create the usage problem filters (requires .settings/.api_filters to be checked in) and the better way is for CDT (and others) to mark this provisional API that has been provisional for 10+ years to be no longer provisional anymore!
  • lots of warnings about progress monitors - to fix this requires moving to the newer progress monitor API. There is quite a good write up on the new API and how to migrate here https://www.eclipse.org/articles/Article-Progress-Monitors/article.html
  • Unknown referenced nature: org.eclipse.m2e.core.maven2Nature - This one happens because I don't follow https://eclipse-embed-cdt.github.io/develop/build-prerequisites/ precisely, specifically I don't use the m2e plug-ins for Eclipse when I am developing Eclipse plug-ins and using Tycho.

That would get down to ~40ish warnings which would be great.

I think the above would make good first issues for new contributors.

jonahgraham avatar Dec 07 '20 15:12 jonahgraham

specifically I don't use the m2e plug-ins for Eclipse when I am developing Eclipse plug-ins and using Tycho.

You do not follow the steps in the build-prerequisites page because they are not correct? If you have a better configuration, we should improve the web page.

Otherwise I think that it would be better for you to have a separate Eclipse and a separate workspace, installed exactly as documented, so we can have a consistent way of spotting errors.

ilg-ul avatar Dec 07 '20 16:12 ilg-ul