Skip to content

docs: add mandatory header workflow reminder to Licensing Rules#1078

Merged
He-Pin merged 1 commit into
mainfrom
docs/licensing-header-reminder
Jun 19, 2026
Merged

docs: add mandatory header workflow reminder to Licensing Rules#1078
He-Pin merged 1 commit into
mainfrom
docs/licensing-header-reminder

Conversation

@He-Pin

@He-Pin He-Pin commented Jun 19, 2026

Copy link
Copy Markdown
Member

Motivation

The existing Licensing Rules already state "Do not hand-write or invent license headers. Let sbt manage them" and "For new files, run sbt headerCreateAll", but agents can still miss these rules and hand-write a header (e.g. accidentally using the Akka-derived header for new code).

Modification

Add a prominent blockquote reminder at the top of the Licensing Rules section that spells out the mandatory workflow in one line: create file without header → sbt headerCreateAll → done.

Result

Agents see the workflow before reading the detailed rules, reducing the chance of hand-writing an incorrect header.

Tests

Not run - docs only

References

None - process improvement

Motivation:
Agents can miss the rule about not hand-writing license headers and
accidentally paste the wrong header (e.g. the Akka-derived header for
new code).

Modification:
Add a prominent blockquote reminder at the top of the Licensing Rules
section that spells out the mandatory workflow: create file without
header -> sbt headerCreateAll -> done.

Result:
Agents see the workflow before reading the detailed rules, reducing
the chance of hand-writing an incorrect header.

Tests:
Not run - docs only

References:
None - process improvement

@pjfanning pjfanning left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@He-Pin He-Pin merged commit a42772b into main Jun 19, 2026
5 checks passed
@He-Pin He-Pin deleted the docs/licensing-header-reminder branch June 19, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants