Health workflow: Mono repo commit message format #332

Activity

1. mosuem commented on Jan 2, 2025

   

   mosuem

   on Jan 2, 2025

   Member

   More actions

   We could add this check. I would be hesitant to add this for all dart-lang repos though, and we should probably have that discussion/decision first before doing the implementation work.

   cc @devoncarew

   👍React with 👍1Reacted by Daco Harkes
2. devoncarew commented on Jan 2, 2025

   

   devoncarew

   on Jan 2, 2025

   Contributor

   More actions

   Hmm, we could dogfood something like this w/ an off-the shelf tool before doing something like adding it to the health checks. This could be via a git hook, or a github action (ala https://github.com/marketplace/actions/gs-commit-message-checker).
3. devoncarew commented on Jan 2, 2025

   

   devoncarew

   on Jan 2, 2025

   Contributor

   More actions

   fwiw, I don't have a particular opinion about using package prefixes for commit messages, but haven't really given it any thought.
4. dcharkes commented on Jan 2, 2025

   

   dcharkes

   on Jan 2, 2025

   ContributorAuthor

   More actions

   Hmm, we could dogfood something like this w/ an off-the shelf tool before doing something like adding it to the health checks. This could be via a git hook, or a github action (ala https://github.com/marketplace/actions/gs-commit-message-checker).

   Dogfooding on dart-lang/native as per:

   • [infra] Add PR title checker native#1856
5. sigurdm

   added

   type-questionA question about expected behavior or functionalityA question about expected behavior or functionality

   on Nov 20, 2025