Skip to content

jcolors.jl: remove LICENSE notice (close #146) #147

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
Feb 12, 2025

Conversation

StefanKarpinski
Copy link
Contributor

Color schemes are not eligible for copyright protection, so the Julia logo license does not apply here. If there's any concern, I am also the copyright owner for the Julia logo.

Color schemes are not eligible for copyright protection, so the Julia logo license does not apply here. If there's any concern, I am also the copyright owner for the Julia logo.
Copy link

codecov bot commented Feb 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.14%. Comparing base (096ea3c) to head (9bc4077).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #147   +/-   ##
=======================================
  Coverage   51.14%   51.14%           
=======================================
  Files           1        1           
  Lines         131      131           
=======================================
  Hits           67       67           
  Misses         64       64           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@StefanKarpinski StefanKarpinski merged commit a2dba9a into master Feb 12, 2025
9 checks passed
@StefanKarpinski StefanKarpinski deleted the sk/rm-jcolors-license branch February 12, 2025 16:49
@GunnarFarneback
Copy link

For this change to really be effective I think a version bump and registration is needed.

@cormullion
Copy link
Member

Yes I will do this in the next x hours, where x is a small-ish integer. 😀

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.

3 participants