You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
All the samples use the same package space. This is generally not a good idea. In order to fix coveralls so it works during github actions, all the samples need to be put into their own name spaces.
This can be seen by simply running this dummy run locally to emulate the issue. Then go into any sample and the issue goes away.
mvn test jacoco:report coveralls:report -q -D"license.skip=true" -DrepoToken=x -DserviceName=github
Note the above run is not intended to work, just to show the issue then show that isolated samples do not have the problem. If you look at the file in error, it happens in all samples and the lines different in each. Most tools would already complain of this approach type.
Solution would be to add in the type of the package. So sample.mybatis for example should become sample.freemarker.mybatis if in the freemarker package.
The text was updated successfully, but these errors were encountered:
@hazendaz I am trying to work on this as good first issue , Can you please validate if changes all sample packages from sample.mybatis to sample.freemarker or sample.groovy etc ... will fix the package issue ? Also I changed test package accordingly sample.annotation.test.mybatis.
Hi @snagapuriosc, I had completely lost track of this. That would do it though. Basic issue is the overlaping naming that it has issues with. By splitting those out the names won't overlap any longer. I forget all the exact details of this issue now but nothing was ever done and I think the instructions I left were enough to address. I probably intended that I did that at some point and was a place holder to not forget but have at it as I see you started playing around with that already.
As discussed to get started on open source contribution I worked on open issue,Can you guys please review and advise on below commit id before I create a PR. snagapuriosc@4ab48cc
All the samples use the same package space. This is generally not a good idea. In order to fix coveralls so it works during github actions, all the samples need to be put into their own name spaces.
This can be seen by simply running this dummy run locally to emulate the issue. Then go into any sample and the issue goes away.
Note the above run is not intended to work, just to show the issue then show that isolated samples do not have the problem. If you look at the file in error, it happens in all samples and the lines different in each. Most tools would already complain of this approach type.
Solution would be to add in the type of the package. So sample.mybatis for example should become sample.freemarker.mybatis if in the freemarker package.
The text was updated successfully, but these errors were encountered: