Skip to content

Don't call --strict-optional and --incremental experimental #4642

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

Merged
merged 3 commits into from
Feb 27, 2018

Conversation

msullivan
Copy link
Collaborator

Also stop documenting strict_optional_whitelist, since it has been
long since obsoleted, with the idea of removing it soon.

Also stop documenting strict_optional_whitelist, since it has been
long since obsoleted, with the idea of removing it soon.
@msullivan msullivan requested a review from gvanrossum February 26, 2018 23:56
- ``--incremental`` enables a module cache, using results from
previous runs to speed up type checking. Incremental mode can help
when most parts of your program haven't changed since the previous
mypy run. A companion flag is ``--cache-dir DIR``, which specifies
Copy link
Member

Choose a reason for hiding this comment

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

I think it's better to make --cache-dir a separate bullet point.

Copy link
Member

@emmatyping emmatyping left a comment

Choose a reason for hiding this comment

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

LGTM. Just one suggestion.

- ``--strict-optional`` enables strict checking of ``Optional[...]``
types and ``None`` values. Without this option, mypy doesn't
generally check the use of ``None`` values -- they are valid
everywhere. See :ref:`strict_optional` for more about this feature.
Copy link
Member

Choose a reason for hiding this comment

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

Minor suggestion: depending on how soon, you might want to mention this flag will become default soon.

Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

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

(But if you could also add the remark suggested by Ethan in his review that would be great.)

@msullivan msullivan merged commit 064c8d3 into master Feb 27, 2018
@msullivan msullivan deleted the doc-updates branch February 27, 2018 01:16
carljm added a commit to carljm/mypy that referenced this pull request Feb 28, 2018
* master: (27 commits)
  Don't call --strict-optional and --incremental experimental (python#4642)
  Sync typeshed (python#4641)
  Fix callable types with inconsistent argument counts (python#4611)
  Fix example (add 'class A:')
  Make psutil an optional dependency (python#4634)
  mypy and mypy_extensions aren't posix only (python#3765)
  Documentation for attr support (python#4632)
  Use read_with_python_encoding in stubgen to handle file encoding (python#3790)
  Sync typeshed (python#4631)
  Add remaining core team emails to CREDITS (python#4629)
  Fix issues with attr code. (python#4628)
  Better support for converter in attrs plugin. (python#4607)
  Clean up credits (python#4626)
  Support type aliases in fine-grained incremental mode (python#4525)
  Fine-grained: Fix crash caused by unreachable class (python#4613)
  Treat divmod like a binary operator (python#4585)
  Sync typeshed (python#4605)
  Fine-grained: Don't infer partial types from multiple targets (python#4553)
  Fine-grained: Compare symbol table snapshots when following dependencies (python#4598)
  Fix type of forward reference to a decorated class method (python#4486)
  ...
yedpodtrzitko pushed a commit to kiwicom/mypy that referenced this pull request Mar 15, 2018
)

Also stop documenting strict_optional_whitelist, since it has been
long since obsoleted, with the idea of removing it soon.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants