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 license #4257

Merged
merged 2 commits into from
Nov 6, 2024
Merged

Update license #4257

merged 2 commits into from
Nov 6, 2024

Conversation

paemurru
Copy link
Contributor

No description provided.

LICENSE.md Outdated Show resolved Hide resolved
Co-authored-by: Max Horn <[email protected]>
Copy link

codecov bot commented Oct 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.59%. Comparing base (7769ffa) to head (900e559).
Report is 68 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4257      +/-   ##
==========================================
+ Coverage   84.58%   84.59%   +0.01%     
==========================================
  Files         631      640       +9     
  Lines       84829    85116     +287     
==========================================
+ Hits        71754    72008     +254     
- Misses      13075    13108      +33     

see 64 files with indirect coverage changes

> See <https://www.oscar-system.org/contributors/> for a list of OSCAR
> Development Team members. Note that this list may occasionally be slightly
> outdated. Additional contributors may be found in the `git log` of the main
> OSCAR repository and other related repositories.
Copy link
Member

Choose a reason for hiding this comment

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

Also relevant: #277 -- perhaps that can be closed with this PR (and with the work by @HereAround and @aaruni96 on oscar-system/oscar-website#373)

Copy link
Member

@fingolfin fingolfin left a comment

Choose a reason for hiding this comment

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

Sounds good to me, thank you.

@fingolfin
Copy link
Member

I'll wait a bit and see if @thofma or @fieker or someone else has further comments and will otherwise merge this on Friday or so

@thofma
Copy link
Collaborator

thofma commented Oct 30, 2024

if @fingolfin is happy, I am happy

@fingolfin fingolfin merged commit 953ad23 into oscar-system:master Nov 6, 2024
29 checks passed
@paemurru paemurru deleted the ep/update_license branch November 6, 2024 12:36
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