Skip to content

Add SPDX headers for file.license_id (beman-tidy)#15

Merged
neatudarius merged 1 commit into
bemanproject:mainfrom
neatudarius:issue/251-file-license-id
Jul 9, 2026
Merged

Add SPDX headers for file.license_id (beman-tidy)#15
neatudarius merged 1 commit into
bemanproject:mainfrom
neatudarius:issue/251-file-license-id

Conversation

@neatudarius

Copy link
Copy Markdown
Member

Issue: bemanproject/beman-tidy#251

Problem

beman-tidy default mode fails on file.license_id — 7 files missing SPDX headers.

Changes

Add # SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception as the first line in:

  • .github/workflows/pre-commit.yml
  • .markdownlint.yaml
  • .pre-commit-config.yaml
  • cmake/beman.bounds_test-config.cmake
  • cmake/bootstrap_vcpkg.cmake
  • cmake/check_plat.cmake
  • include/beman/bounds_test/plat/CMakeLists.txt

beman-tidy results (local, default mode)

Summary    Requirement:  16 checks passed, 0 checks failed, 3 checks skipped,  8 checks not implemented.

Still failing with --require-all:

  • release.godbolt_trunk_version — no Compiler Explorer badge in README

Test plan

  • beman-tidy . --verbose — 0 requirement failures
  • CI pre-commit check passes

Issue: bemanproject/beman-tidy#251

Add Apache-2.0 WITH LLVM-exception headers to config, workflow,
and cmake files so beman-tidy default mode passes file.license_id.

Co-authored-by: Cursor <cursoragent@cursor.com>
@neatudarius
neatudarius marked this pull request as ready for review July 9, 2026 19:46
@neatudarius
neatudarius merged commit c339c46 into bemanproject:main Jul 9, 2026
2 checks passed
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.

1 participant