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

Comply with Adobe Legal #43

Merged

Conversation

adifucan
Copy link
Contributor

  • Add license;
  • Add code of conduct;
  • Add contribution guidelines;
  • Add copyright header;

- Add license;
- Add code of conduct;
- Add contribution guidelines;
- Add copyright header;
@DrewML
Copy link
Contributor

DrewML commented Jan 31, 2020

@filmaj does Adobe have an existing package we can use in CI to validate new files have the licensing requirements going forward? I have an adhoc script, but something maintained might be better if it exists

@filmaj
Copy link

filmaj commented Feb 3, 2020

We do not maintain something like that but there are existing tools out there: Apache RAT for Apache-2.0 licensed projects is a common one, and depending on your language, usually there are lint plugins available for ensuring copyright header are present. For JS:


For more detailed information on contribution please read our [beginners guide](https://github.com/magento/magento2/wiki/Getting-Started).

## Contribution requirements
Copy link
Contributor

Choose a reason for hiding this comment

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

We should change these contribution requirements, since we're not enforcing quite a few of the standards required in Magento core.

Some notes:

  1. Docs go in this repo, rather than devdocs (want them versioned together with code)
  2. We don't require following Magento coding standards

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Contribution requirements updated.

- Update contribution guidelines;
@DrewML DrewML merged commit f6dc98d into magento:master Feb 3, 2020
@DrewML
Copy link
Contributor

DrewML commented Feb 3, 2020

Thanks @adifucan / @filmaj!

@DrewML
Copy link
Contributor

DrewML commented Feb 3, 2020

Opened #46 to track automation of license check

@filmaj
Copy link

filmaj commented Feb 3, 2020

Thanks for the speedy resolution here y'all <3

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