Skip to content
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

update copyright year to 2025 #1924

Merged
merged 2 commits into from
Feb 24, 2025
Merged

update copyright year to 2025 #1924

merged 2 commits into from
Feb 24, 2025

Conversation

wgyt
Copy link
Contributor

@wgyt wgyt commented Feb 20, 2025

No description provided.

@wgyt wgyt requested a review from a team as a code owner February 20, 2025 03:15
@cla-bot-2021
Copy link

cla-bot-2021 bot commented Feb 20, 2025

User @wgyt, please sign the CLA here.

@wgyt
Copy link
Contributor Author

wgyt commented Feb 20, 2025

User @wgyt, please sign the CLA here.

CLA signed.

@niklasad1
Copy link
Member

@CodiumAI-Agent /review

@CodiumAI-Agent
Copy link

CodiumAI-Agent commented Feb 24, 2025

PR Reviewer Guide 🔍

(Review updated until commit bab4c97)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@@ -1,3 +1,3 @@
// Copyright 2019-2023 Parity Technologies (UK) Ltd.
// Copyright 2019-2025 Parity Technologies (UK) Ltd.
Copy link
Collaborator

@lexnv lexnv Feb 24, 2025

Choose a reason for hiding this comment

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

dq: Maybe we can remove the years entirely? And do a similar update to polkadot-sdk?

// This file is part of Subxt.

// Copyright (C) Parity Technologies (UK) Ltd.

...

Copy link
Member

Choose a reason for hiding this comment

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

would be better, I don't remember why the year is needed in the copywright header but "not a laywer" :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yup agreed this would be better, though for now I'll just merge this PR to avoid looking into whether it matters at all :)

@jsdw jsdw merged commit 3967625 into paritytech:master Feb 24, 2025
13 checks passed
@CodiumAI-Agent
Copy link

Persistent review updated to latest commit bab4c97

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.

5 participants