Skip to content

Workflow harmonization (CI housekeeping)#1

Merged
michaldengusiak merged 1 commit into
sow/2026-Q2from
feature/remove-newtonsoft
May 18, 2026
Merged

Workflow harmonization (CI housekeeping)#1
michaldengusiak merged 1 commit into
sow/2026-Q2from
feature/remove-newtonsoft

Conversation

@michaldengusiak

@michaldengusiak michaldengusiak commented May 18, 2026

Copy link
Copy Markdown

Summary

Housekeeping commit that didn't make it into the original PR set because this repo had no Newtonsoft source code to migrate (ahead=0 at the time PRs were opened). The workflow harmonization sweep added 1–2 commits to feature/remove-newtonsoft afterward, which now need merging.

Contents:

  • Aligns .github/workflows/spdx-check.yml with the org-canonical template used by every other SAM-BIM repo (lenient [-–] dash matching, 20-line lookback, BOM/CR-aware).
  • For SAM_Validation only: bumps the expected copyright year from 2020–2025 to 2020–2026.

No source code changes. No build behaviour changes.

Test plan

  • CI build (if present) stays green
  • SPDX check fires correctly on subsequent PRs

🤖 Generated with Claude Code

Replaces the older spdx-check.yml (literal en-dash grep, top 6 lines,
PR-only trigger) with the modern template shipped on SAM core:
- grep -qE with [-–] char class — accepts both hyphen-minus and en-dash
- 20-line lookback (was 6)
- BOM and CR stripping
- mapfile + diff-filter for cleaner change detection
- Adds workflow_dispatch trigger

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@michaldengusiak
michaldengusiak merged commit df610e6 into sow/2026-Q2 May 18, 2026
1 check passed
@michaldengusiak
michaldengusiak deleted the feature/remove-newtonsoft branch May 18, 2026 13:25
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