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

⚖️♻️:refactor licenses to canonical location #643

Merged
merged 2 commits into from
Dec 20, 2023

Conversation

DerekNonGeneric
Copy link
Member

Pull Request Purpose

Make a selection on the purpose(s) of your changes.

This PR contains the following:

  • 🐞🔧 bugfixing (🐜/🦟/🐛/🦗/🐝 et al.)
  • 🆕🎏 implementation of new feature(s)
  • ♻️ refactoring(s)
  • 📄 documentation modification(s)
  • 🔮 other

Testing

Have you included any tests in your submission?

  • yes (bugfixes and features will not be merged without
    tests
    )
  • no

Breaking Changes

Have you modified any aspect public API surface (i.e., caused breakage)?

  • yes (breaking changes will not be merged unless
    necessary
    )
  • no

Description

A concise recap summarily detailing what all are the changes being made.

Self-explanatory, see the associated issue (shouldn't close it tho).

List of any relevant issue numbers:

@@ -1,16 +0,0 @@
## License
Copy link
Member Author

Choose a reason for hiding this comment

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

Seems like we should move all this to our README since it does seem relevant…

See bottom section: https://github.com/OpenINF/.github/blob/main/README.md#contributing

@@ -1,23 +0,0 @@
Permission is hereby granted, free of charge, to any
Copy link
Member Author

Choose a reason for hiding this comment

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

i did some snooping and this file (decapitated and weirdly small) originally came from the Rust project…

See: https://github.com/rust-lang/rust/blob/51c0db6a9118735190466d15432b5d8890bb09ee/LICENSE-MIT

i think we might need to go headless horseman on ours, too!

LICENSE/MIT.txt Outdated Show resolved Hide resolved
@DerekNonGeneric DerekNonGeneric added the author-ready PRs that are considered ready by their author label Dec 20, 2023
@OpenINFbot OpenINFbot added the fast-track PRs that do not need to wait for 48 hours to land label Dec 20, 2023
@DerekNonGeneric DerekNonGeneric merged commit 2a15e2d into OpenINF:main Dec 20, 2023
4 checks passed
@DerekNonGeneric DerekNonGeneric deleted the fix/license-dir branch December 20, 2023 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author-ready PRs that are considered ready by their author fast-track PRs that do not need to wait for 48 hours to land
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants