grails-data-mapping icon indicating copy to clipboard operation
grails-data-mapping copied to clipboard

Published 20 hours ago • verified publisher icon grails

The PR to merge changes to grails data-mapping to apply @Generated to genrated methods has not been merged to version used by 5.1.7

Open boardbloke opened this issue 3 years ago • 3 comments

There is a PR submitted and merged to the grails data-mapping plugin to apply @Generated to generated fields. This makes covrag tools like jacoco usable. Without ths change the code coverage report contains a lot of cruft which makes it pretty unusable.

the PR is https://github.com/grails/grails-data-mapping/pull/1507 the original issue is https://github.com/grails/grails-data-mapping/issues/1445

Can it be merged to version of the plugin being used by Grails 5.1.7+ please?

boardbloke avatar Jun 17 '22 08:06 boardbloke

HI @puneetbehl - Any chance this could be looked at?

boardbloke avatar Jul 13 '22 16:07 boardbloke

Is this on the roadmap? @puneetbehl

davebrown1975 avatar Feb 08 '23 11:02 davebrown1975

Hi @puneetbehl - Would it possible to get this in release 8 of grails-data-mapping. Its making it difficult to get accurate code coverage metrics.

The commit that contains the merge to 7.0.9 branch is here https://github.com/grails/grails-data-mapping/commit/ed6d4740f2d80d64b78d4aa1ec645c20afe1e3ba

It needs merging into the 8.0 line.

boardbloke avatar Apr 04 '23 10:04 boardbloke