Skip to content

chore(ci): run copywrite after fetch #509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 4, 2025
Merged

Conversation

xiehan
Copy link
Contributor

@xiehan xiehan commented Jun 4, 2025

Companion to cdktf/cdktf-repository-manager#391. This isn't a major issue since the copyright headers do get added eventually (during the self-mutation step as part of the build CI workflow after the PR gets created) but if we know that the headers need to be recreated anyway because fetch nuked them, then we might as well do it up-front.

@xiehan xiehan requested a review from a team as a code owner June 4, 2025 08:50
@xiehan xiehan removed the request for review from a team June 4, 2025 08:50
@xiehan xiehan requested review from mutahhir and DanielMSchmidt June 4, 2025 08:50
@team-tf-cdk team-tf-cdk enabled auto-merge (squash) June 4, 2025 08:50
@team-tf-cdk team-tf-cdk merged commit 7b01a4e into main Jun 4, 2025
11 checks passed
@team-tf-cdk team-tf-cdk deleted the copywrite-after-fetch branch June 4, 2025 08:59
Copy link

I'm going to lock this pull request because it has been closed for at least 7 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue so we can investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants