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.

Sign up for GitHub

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

Jump to bottom

Remove year from copyright #9487

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Remove year from copyright #9487

merged 1 commit into from
Jan 7, 2025

Conversation

def-
Copy link
Member

@def- def- commented Jan 7, 2025 •
edited
Loading

No one cares, useless to bump every year

Just check the git log to see when last changes happened.

Checklist

  • Tested the change ingame
  • Provided screenshots if it is a visual change
  • Tested in combination with possibly related configuration options
  • Written a unit test (especially base/) or added coverage to integration test
  • Considered possible null pointers and out of bounds array indexing
  • Changed no physics that affect existing maps
  • Tested the change with ASan+UBSan or valgrind's memcheck (optional)

@def-
Remove year from copyright
7aa0ff1 
No one cares, useless to bump every year
heinrich5991
heinrich5991 approved these changes Jan 7, 2025
View reviewed changes
Copy link
Member

@heinrich5991 heinrich5991 left a comment

Choose a reason for hiding this comment

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

Thanks.

@heinrich5991 heinrich5991 enabled auto-merge January 7, 2025 12:38
@heinrich5991 heinrich5991 added this pull request to the merge queue Jan 7, 2025
@ChillerDragon
Copy link
Contributor

After 10 years of quality year bumping we finally reached the epic end.

@ChillerDragon
Copy link
Contributor

@VoxelDoesCode mentioned on discord that keeping the start year might be a good idea. I agree. Finding the start is harder than finding the end and it is also no maintenance.

Merged via the queue into ddnet:master with commit b17d6ed Jan 7, 2025
17 checks passed
@heinrich5991
Copy link
Member

Sounds good to me. Maybe "2013-now"?

@Chairn
Copy link
Contributor

Chairn commented Jan 7, 2025

People reading the file in the year 2345

Damn, it's still copyrighted by this vampire

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@heinrich5991 heinrich5991 heinrich5991 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@def- @ChillerDragon @heinrich5991 @Chairn