chore: add SPDX license headers to project-info.conf and paradox .htaccess - #1306
Merged
pjfanning merged 1 commit intoSep 17, 2026
Merged
Conversation
…ccess Motivation: A scan of the source tree for files without an Apache license header found two that sbt-header never sees: `project/project-info.conf`, which lives outside every source set, and `docs/src/main/paradox/.htaccess`, which has no sbt-header file type. Mirrors apache/pekko#3545, which added headers to the logback XML files, shell scripts and benchmark template that sbt-header misses in pekko. pekko-http has no logback configs, its one shell script already carries a header and its boilerplate templates are covered by the `*.scala.template` mapping in `CopyrightHeader`, so only these two files remain. Modification: Prepend the `# SPDX-License-Identifier: Apache-2.0` line that `CopyrightHeader.hashLineComment` hardcodes for every hash-comment file in this repo, matching the header on the `reference.conf` and `application.conf` files and on pekko's own `project-info.conf`. Result: Every tracked configuration and script file carries a license header. No behavior change: HOCON and Apache httpd both treat `#` lines as comments. Tests: - `project/project-info.conf` parsed and resolved with Typesafe Config 1.4.6 after the change; `project-info.scaladoc` resolves as before. - sbt headerCheckAll - pass. References: None - follows apache/pekko#3545
Philippus
approved these changes
Sep 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
A scan of the source tree for files without an Apache license header found two that sbt-header never sees:
project/project-info.conf, which lives outside every source set, anddocs/src/main/paradox/.htaccess, which has no sbt-header file type.Mirrors apache/pekko#3545, which added headers to the logback XML files, shell scripts and benchmark template that sbt-header misses in pekko. pekko-http has no logback configs, its one shell script already carries a header and its boilerplate templates are covered by the
*.scala.templatemapping inCopyrightHeader, so only these two files remain.Modification
Prepend the
# SPDX-License-Identifier: Apache-2.0line thatCopyrightHeader.hashLineCommenthardcodes for every hash-comment file in this repo, matching the header on thereference.confandapplication.conffiles and on pekko's ownproject-info.conf.Result
Every tracked configuration and script file carries a license header. No behavior change: HOCON and Apache httpd both treat
#lines as comments.Tests
project/project-info.confparsed and resolved with Typesafe Config 1.4.6 after the change;project-info.scaladocresolves as before.sbt headerCheckAll- pass.References
None - follows apache/pekko#3545