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 and contributing #539

Merged
merged 4 commits into from
May 16, 2024
Merged

Conversation

nwlandry
Copy link
Collaborator

Made some small updates:

  • Added 2024 to the license files
  • Removed now-unnecessary acknowledgments in the license file
  • Simplified contribution guide.

Copy link

codecov bot commented May 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.35%. Comparing base (6495dcc) to head (6d12a10).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #539   +/-   ##
=======================================
  Coverage   92.35%   92.35%           
=======================================
  Files          60       60           
  Lines        4500     4500           
=======================================
  Hits         4156     4156           
  Misses        344      344           

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

@thomasrobiglio
Copy link
Collaborator

@nwlandry let me know when you are done and I will approve this.

(Are we removing the HypernetX and NetworkX acknowledgments simply because we no longer have code copied from them or for other reasons? thx)

@nwlandry
Copy link
Collaborator Author

@nwlandry let me know when you are done and I will approve this.

(Are we removing the HypernetX and NetworkX acknowledgments simply because we no longer have code copied from them or for other reasons? thx)

@thomasrobiglio ready for review! At this point, we have rewritten the code that we were using from other libraries (I could be wrong about this) so I thought we could simplify our license.

@thomasrobiglio
Copy link
Collaborator

Good to go! thank you @nwlandry

At this point, we have rewritten the code that we were using from other libraries (I could be wrong about this) so I thought we could simplify our license.

That's what I thought, thank you :)

@nwlandry nwlandry merged commit 9355c38 into main May 16, 2024
24 checks passed
@nwlandry nwlandry deleted the update-docs-and-requirements branch May 16, 2024 15:16
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.

None yet

2 participants