Skip to content

SDKS-5477 Add CI validation for copyright headers - #141

Merged
spetrov merged 2 commits into
mainfrom
SDKS-5477
Oct 7, 2026
Merged

spetrov merged 2 commits into
mainfrom
SDKS-5477

Conversation

@spetrov

@spetrov spetrov commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

JIRA Ticket

SDKS-5477 Add CI validation for copyright headers

Description:

Adds a CI check that validates the Ping Identity copyright header of the files changed by a PR. It is the same check as the ones recently added to ping-android-sdk and ping-ios-sdk...

What it does

  • Runs on every pull_request, with no path filter, so Android and iOS changes are covered too. The existing CI.yml and flutter-ci.yml only trigger for javascript/** and flutter/**.
  • Checks only the files added, copied, modified or renamed by the PR, by diffing against the PR base SHA.
  • Fails the check if a file has no Ping Identity copyright header.
  • Fails the check if the header's end year is earlier than the current year. For example, 2024 - 2025 is stale in 2026.

Changes

  • .github/scripts/check-copyright.sh: checks .kt, .java, .swift, .dart, .js and .jsx files for correct copyright header.
  • .github/workflows/copyright-check.yml: new workflow that runs the script with the PR base SHA and repo.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: ForgeRock/sdk-sample-apps/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6112d173-abc8-4468-8a8f-857321121733
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@rodrigoareis rodrigoareis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@spetrov
spetrov merged commit ff111cb into main Oct 7, 2026
2 of 4 checks passed
@spetrov
spetrov deleted the SDKS-5477 branch October 7, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants