sqlalchemy / sqlalchemy/mako

Clarify license in _ast_util.py

Open
#172 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

documentation low priority task
Dominant language
Python
Stars
459
Forks
90
PR merge metrics
No merged PRs in 30d

Description

Migrated issue, originally created by Anonymous

mako/_ast_util.py has the standard mako header at the top listing copyright holder as the mako authors and license terms as MIT. But the docstring that immediately follows says that the copyright holder is Armin Ronacher and license is the Python License.

These are compatible so it's not a big deal whatever permutation this turns out to be but it would be great to get clarification in the header of whether this file is actually under the Python or MIT license or both.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Read mako/_ast_util.py, especially its standard Mako header and the immediately following docstring. Confirm which license applies to the file with the project context, then make the header and docstring consistent so the applicable license is unambiguous.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
documentation
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.