matrei commented on PR #16726: URL: https://github.com/apache/grails-core/pull/16726#issuecomment-6343531428
Thanks both. @codeconsole, you're right about the cost: on a run without tests, everything after `publishGradle` moves back by the same amount. I've corrected the cost section of the description with the numbers from 39615263956 and 39644767954, and added a "Merging up" section. @jdaugherty, the drafts point isn't about publishing, sorry for the confusion. On a draft, `run ci` does two things: it lets the draft run at all, and it asks for the tests. So a draft into 7.0.x, or into a side branch, gets either no CI or the full test matrix. It never gets the build-only run it gets once it's marked ready. Before #16673 every draft ran the full matrix, so nothing is lost compared with that. If that's how you want it, I'm fine leaving it. On `[skip tests]`, @codeconsole's reply covers what I meant. A squash merge of a single-commit PR, or a rebase merge, keeps the commit's title. And on a fork PR into 8.0.x, the PR's own tests never run. Happy to leave that for the weekly. -- 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]
