Skip to content
This repository has been archived by the owner on Nov 22, 2021. It is now read-only.

fix: SPDX license identifier and missing peer dependencies #61

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

sambacha
Copy link

SPDX

this pull request changes // SPDX-License-Identifier: MIT to reflect the newer license of /// SPDX-License-Identifier: SSPL-1.0

note - /// usage for solc warnings even tho

AGPL vs SSPL

See a diff here: AGPL_vs_SSPL

tl;dr SSPL addresses issues of SaaS usage, mainly distributing the software over a network and not disclosing interfaces, etc

Previous PR

#47 and #60
closed previous PR, #47 to address styling changes

Notes

in the future, there should be a 'dummy' git account in which automated processes (such as CI/CD/etc) can be assigned to, e.g. git config user.email git@iearn.finance in order to assist in the review process / diff'ing / etc, esp. considering increasing amounts of outside contributors, and other automated services (ci pipelines, etc)

yEarn Bot

see https://github.com/yearnbot

example for usage in prettier: https://github.com/yearnBot/vcs/tree/main/bin

I did not use this in this PR as there is no point

Optional Header

Copyright 2020 - yearn protocol contributors  

This program is free software: you can redistribute it
and/or modify it under the terms of the Server Side Public
License, version 1, as published by MongoDB, Inc.
See the Server Side Public License for more details. You should
have received a copy of the Server Side Public License
along with this program.

<http://www.mongodb.com/licensing/server-side-public-license>.

This program is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty
of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. 

sam added 2 commits October 24, 2020 19:45
this changes agpl to sspl license

pr-47
this fixes an issue in pr yearn#60

fix pr yearn#60
@saltyfacu
Copy link

Is this still relevant?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants