jdaugherty commented on code in PR #14809: URL: https://github.com/apache/grails-core/pull/14809#discussion_r2145062037
########## grails-doc/src/en/guide/REST/angularJsProfile.adoc: ########## @@ -21,6 +21,8 @@ Since Grails 3.1, Grails supports a profile for creating applications with Angul To get started with the AngularJS profile, create an application specifying `angularjs` as the name of the profile: +// Reviewed — no changes needed Review Comment: I call these "this is bridge" comments. They are self obvious at this point. Let's remove the redundant comment? It's there historically in git if anyone is curious. ########## grails-doc/src/en/guide/REST/angularProfile.adoc: ########## @@ -23,6 +23,18 @@ The biggest change in this profile is that the profile creates a multi project g To get started with the Angular profile, create an application specifying `angular` as the name of the profile: +//NEW Review Comment: Unnecessary comment, the diff will show us new items. It looks like this is a pattern that's been used. Can you please remove all of these so I can redo this review after? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
