jdaugherty commented on code in PR #15003: URL: https://github.com/apache/grails-core/pull/15003#discussion_r2292434356
########## grails-gsp/core/README.md: ########## @@ -31,4 +31,4 @@ apply plugin: "org.apache.grails.gradle.grails-gsp" Dependencies ----- -To see what additional subprojects will be included with this, you can view this project's [build.gradle](https://github.com/grails/grails-core/blob/master/grails-gsp/build.gradle) +To see what additional subprojects will be included with this, you can view this project's [build.gradle](https://github.com/apache/grails-core/blob/HEAD/grails-gsp/plugin/build.gradle) Review Comment: Should this be HEAD? ########## grails-fields/grails-app/taglib/grails/plugin/formfields/FormFieldsTagLib.groovy: ########## @@ -560,7 +560,7 @@ class FormFieldsTagLib { def innerAttributes = attributes.clone() innerAttributes.remove('bean') innerAttributes.remove('prefix') - //'except' is a reserved word for the 'all' tag: https://github.com/grails-fields-plugin/grails-fields/issues/12 + //'except' is a reserved word for the 'all' tag Review Comment: Shouldn't we leave the original ticket link, but fix it? -- 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: notifications-unsubscr...@grails.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org