Skip to content

Update license information in pyproject.toml#1776

Merged
bcantoni merged 3 commits into
tableau:developmentfrom
ecederstrand:patch-1
Jun 4, 2026
Merged

Update license information in pyproject.toml#1776
bcantoni merged 3 commits into
tableau:developmentfrom
ecederstrand:patch-1

Conversation

@ecederstrand
Copy link
Copy Markdown
Contributor

Fixes license specifiers according to https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license-and-license-files

This allows automatic license inspection tools like pip-licenses to correctly categorize this package.

Fixes license specifiers according to https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license-and-license-files

This allows automatic license inspection tools like pip-licenses to correctly categorize this package.
@salesforce-cla
Copy link
Copy Markdown

Thanks for the contribution! Before we can merge this, we need @ecederstrand to sign the Salesforce Inc. Contributor License Agreement.

@ecederstrand
Copy link
Copy Markdown
Contributor Author

Signed

@jacalata
Copy link
Copy Markdown
Contributor

(close and re-open to kick the cla-bot)

@jacalata
Copy link
Copy Markdown
Contributor

install fails with

raise TOMLDecodeError(

pip._vendor.tomli._parser.TOMLDecodeError: Expected newline or end of document after a statement (at line 11, column 21)

@ecederstrand
Copy link
Copy Markdown
Contributor Author

Sorry! Should be fixed now.

Copy link
Copy Markdown
Contributor

@jacalata jacalata left a comment

Choose a reason for hiding this comment

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

Our existing license file says the project is under MIT, this probably should too

@ecederstrand
Copy link
Copy Markdown
Contributor Author

So sorry! That was a copy-paste error from a similar PR to another repo. Fixed now.

@ecederstrand
Copy link
Copy Markdown
Contributor Author

@jacalata Is there anything more I need to do for this one to be merged?

@bcantoni bcantoni merged commit 6acd12f into tableau:development Jun 4, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants