BungeeTabListPlus icon indicating copy to clipboard operation
BungeeTabListPlus copied to clipboard

Add Velocity Support

Open proferabg opened this issue 9 months ago • 7 comments

proferabg avatar Feb 15 '25 20:02 proferabg

A few points I want to give my personal input towards.

One main thing that should be done, but that I didn't want to mention on every single file, is to update the copyright header for every added and changed file to have 2025 as the year, since this was the year those files where added/changed and where the copyright becomes effective.

Who should I copyright on this because it certainly wasn't Florian that ported it. Should I copyright with my own name or is there a project setup so that BungeeTabListPlus can be copywritten in itself?

proferabg avatar Feb 16 '25 03:02 proferabg

Who should I copyright on this because it certainly wasn't Florian that ported it. Should I copyright with my own name or is there a project setup so that BungeeTabListPlus can be copywritten in itself?

It should work having both. Like

Copyright (c) (your name) Copyright (c) (btlp dev)

Saw this within other projects.

Andre601 avatar Feb 16 '25 12:02 Andre601

Personally, I find the copyright notices a bit annoying to keep up-to-date. If you want, then you can update them, but I would also be fine merging this as is. Adding your own name, as Andre601 suggests, is fine.

CodeCrafter47 avatar Feb 16 '25 12:02 CodeCrafter47

I was told copyright really is only necessary to update for files that get updated.

At least the newly sdded files should use 2025, as they didn't exis before, no?

Andre601 avatar Feb 16 '25 12:02 Andre601

Copyrights have been updated, the metrics have changed to the new ID for velocity, and I adjusted the Team packet to use an enumeration for modes as well as cleaned up some of the map initializers.

proferabg avatar Feb 16 '25 23:02 proferabg

Is this good to be merged from your side?

CodeCrafter47 avatar Feb 22 '25 09:02 CodeCrafter47

Is this good to be merged from your side?

As long as there are nothing else needing changed and we are good on all the questions, then this can be merged.

proferabg avatar Feb 23 '25 02:02 proferabg

It's merged now. Sorry this took so long.

CodeCrafter47 avatar Mar 09 '25 09:03 CodeCrafter47