Skip to content

Latest commit

 

History

History
156 lines (117 loc) · 5.91 KB

CONTRIBUTING.md

File metadata and controls

156 lines (117 loc) · 5.91 KB
Raw

Contributing to Mypy

welcome! basedmypy is a community project that aims to work for a wide range of Python users and Python codebases. If you're trying basedmypy on your Python code, your experience and what you can contribute are important to the project's success

Getting started with development

Setup

(1) Fork the mypy repository

Within Github, navigate to https://github.com/KotlinIsland/basedmypy and fork the repository.

(2) Clone the mypy repository and enter into it

clone the project using your IDE

if you don't use an IDE, you can also use the terminal (not recommended, the terminal is better left in the 80's):

git clone https://github.com/<your_username>/basedmypy.git
cd basedmypy

(3) Setup the project

./pw install

Note

You'll need Python 3.8 or higher to install all requirements you can specify the version:

./pw install --python 3.13

Running tests

Running the full test suite can take a while, and usually isn't necessary when preparing a PR. Once you file a PR, the full test suite will run on GitHub. You'll then be able to see any test failures, and make any necessary changes to your PR.

However, if you wish to do so, you can run the full test suite like this:

python runtests.py

Some useful commands for running specific tests include:

# Use mypy to check mypy's own code
.\pw typecheck

# Run a single test from the test suite
.\pw test test_name

# Run all test cases in the "test-data/unit/check-dataclasses.test" file
pytest mypy/test/testcheck.py::TypeCheckSuite::check-dataclasses.test

# Run the formatters and linters
.\pw format
.\pw lint

For an in-depth guide on running and writing tests, see the README in the test-data directory.

First time contributors

If you're looking for things to help with, browse our issue tracker!

In particular, look for good first issues

You do not need to ask for permission to work on any of these issues. Just fix the issue yourself, try to add a unit test and open a pull request.

To get help fixing a specific issue, it's often best to comment on the issue itself. You're much more likely to get help if you provide details about what you've tried and where you've looked (maintainers tend to help those who help themselves). gitter can also be a good place to ask for help.

Interactive debuggers like pdb and ipdb are really useful for getting started with the mypy codebase. This is a useful tutorial.

It's also extremely easy to get started contributing to our sister project typeshed that provides type stubs for libraries. This is a great way to become familiar with type syntax.

Submitting changes

Even more excellent than a good bug report is a fix for a bug, or the implementation of a much-needed new feature. We'd love to have your contributions.

We use the usual GitHub pull-request flow, which may be familiar to you if you've contributed to other projects on GitHub. For the mechanics, see our git and GitHub workflow help page, or GitHub's own documentation.

Anyone interested in Mypy may review your code. One of the Mypy core developers will merge your pull request when they think it's ready.

If your change will be a significant amount of work to write, we highly recommend starting by opening an issue laying out what you want to do. That lets a conversation happen early in case other contributors disagree with what you'd like to do or have ideas that will help you do it.

The best pull requests are focused, clearly describe what they're for and why they're correct, and contain tests for whatever changes they make to the code's behavior. As a bonus these are easiest for someone to review, which helps your pull request get merged quickly! Standard advice about good pull requests for open-source projects applies; we have our own writeup of this advice.

Also, do not squash your commits after you have submitted a pull request, as this erases context during review. We will squash commits when the pull request is merged.

You may also find other pages in the Mypy developer guide helpful in developing your change.

Core developer guidelines

Core developers should follow these rules when processing pull requests:

  • Always wait for tests to pass before merging PRs.
  • Use "Squash and merge" to merge PRs.
  • Delete branches for merged PRs (by core devs pushing to the main repo).
  • Edit the final commit message before merging to conform to the following style (we wish to have a clean git log output):
    • When merging a multi-commit PR make sure that the commit message doesn't contain the local history from the committer and the review history from the PR. Edit the message to only describe the end state of the PR.
    • Make sure there is a single newline at the end of the commit message. This way there is a single empty line between commits in git log output.
    • Split lines as needed so that the maximum line length of the commit message is under 80 characters, including the subject line.
    • Capitalize the subject and each paragraph.
    • Make sure that the subject of the commit message has no trailing dot.
    • Use the imperative mood in the subject line (e.g. "Fix typo in README").
    • If the PR fixes an issue, make sure something like "Fixes #xxx." occurs in the body of the message (not in the subject).
    • Use Markdown for formatting.