Skip to content

Rename LICENSE.md to LICENSE#365

Open
tschm wants to merge 9 commits intoplotly:masterfrom
tschm:patch-2
Open

Rename LICENSE.md to LICENSE#365
tschm wants to merge 9 commits intoplotly:masterfrom
tschm:patch-2

Conversation

@tschm
Copy link
Copy Markdown
Contributor

@tschm tschm commented Jul 26, 2025

No description provided.

@tschm
Copy link
Copy Markdown
Contributor Author

tschm commented Jul 26, 2025

Markdownlinter doesn't like License.md files!

Copy link
Copy Markdown
Contributor

@gvwilson gvwilson left a comment

Choose a reason for hiding this comment

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

@ayjayt looks good to me, your call if you want the change

@gvwilson gvwilson added community community contribution fix fixes something broken P2 needed for current cycle labels Aug 1, 2025
Copilot AI review requested due to automatic review settings May 2, 2026 18:26
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Python package metadata to reference a non-markdown license file and adds the corresponding LICENSE file content under src/py, aligning packaging metadata with the intended filename.

Changes:

  • Update pyproject.toml to use license = { file = "LICENSE" } instead of referencing LICENSE.md.
  • Add src/py/LICENSE containing the MIT license text.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
src/py/pyproject.toml Points project metadata at LICENSE rather than LICENSE.md.
src/py/LICENSE Introduces the LICENSE file content in the Python package directory.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community community contribution fix fixes something broken P2 needed for current cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants