-
Notifications
You must be signed in to change notification settings - Fork 5
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
Issue/227/include license #228
Conversation
Include the authors in the same way Michelle used in 2018
Merge branch 'issue/227/include-license' of github.com:LSSTDESC/snmachine into issue/227/include-license
@MichelleLochner, when you have time, can you take a look at this? Basically, I added a license, so see if you like it or want to change it/ its wording. Importantly, I commented out some failing tests in |
Solves #227. I added the authors the same way @MichelleLochner did in 2018.
Do you prefer different wording?
Regarding the date, I followed the Guidelines for DESC Repositories (download from here) refers