search

YiNNx / cmd-wrapped

👩‍💻 A CLI Tool for Command Line Insights
https://crates.io/crates/cmd-wrapped
MIT License
1.01k stars 27 forks source link

chore: rename license file #26

Closed xj63 closed 9 months ago

xj63 commented 9 months ago

It may be more popular to use LICENSE files.

By default, this link will jump to the file starting with LICENSE. There is a LICENSE link on the right side of github. https://github.com/YiNNx/cmd-wrapped#MIT-1-ov-file

Some tools also only use LICENSE* files, such as cargo-aur.

The github documentation also recommends using LICENSE file names in all uppercase letters. https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/licensing-a-repository#determining-the-location-of-your-license

xj63 commented 9 months ago

Sorry, the link https://github.com/YiNNx/cmd-wrapped#MIT-1-ov-file cannot jump to the LICENSE interface. It may be my personal problem. License is also valid.

xj63 commented 9 months ago

I'm sorry for my reckless behavior. The License is usually valid and there is no need to change it to LICENSE. It may be better to keep using the License.

By the way, aur already has the cmd-wrapped package.

YiNNx commented 9 months ago

Apologies for the delayed response (have been struggling with some personal matters :( I believe renaming the license filename to LICENSE will be a better option, and I simply hadn't noticed the specification earlier. I'll reopen this PR and merge it, thanks for your suggestions!

YiNNx commented 9 months ago

By the way, aur already has the cmd-wrapped package.

Noticed the aur package page about 2 weeks ago XD. If that was your doing, I really appreciate it!

xj63 commented 9 months ago

Noticed the aur package page about 2 weeks ago XD. If that was your doing, I really appreciate it!

Sorry, I am not the author. The author is devome. I don’t know his contact information. Maybe after you update github release, devome will help you update aur after seeing it. : )