TelluIoT / ThingML

The ThingML modelling language
https://github.com/TelluIoT/ThingML
Apache License 2.0
101 stars 32 forks source link

Workaround comment exists in language/thingml/pom.xml file while the related issues are already fixed. #300

Open markxiaotao opened 3 years ago

markxiaotao commented 3 years ago

In a research project, we analyzed the build file of repositories looking for comments with ready-to-be-addressed SATDs (self-admitted technical debt) that could be addressed. If the underlying issue is already resolved, the related code or comment can be removed or fixed. As we found an instance of this kind of asynchrony in language/thingml/pom.xml, we decided to report the issue to contribute community. A comment in language/thingml/pom.xml file claims that it is workaround for https://github.com/eclipse/xtext/issues/1231, and this issue has been closed long ago, they also provided new solution: https://github.com/eclipse/xtext/issues/1231#issuecomment-460184629.

Comment location: https://github.com/TelluIoT/ThingML/blob/c2d2df93062478675eba8a28b33472461973b0e5/language/thingml/pom.xml#L88