diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 49f45ed2..2aecdc6d 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -33,14 +33,13 @@ repos:
hooks:
- id: black
- - repo: https://github.com/PyCQA/flake8
- rev: 6.0.0
+ - repo: https://github.com/charliermarsh/ruff-pre-commit
+ rev: v0.0.270
hooks:
- - id: flake8
- additional_dependencies: [flake8-bugbear~=22.7]
+ - id: ruff
- repo: https://github.com/pre-commit/mirrors-mypy
- rev: v1.2.0
+ rev: v1.3.0
hooks:
- id: mypy
additional_dependencies: [mdurl]
diff --git a/docs/conf.py b/docs/conf.py
index e0a6e621..6a6ee557 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -121,7 +121,7 @@ def run_apidoc(app):
shutil.rmtree(api_folder)
os.mkdir(api_folder)
- argv = ["-M", "--separate", "-o", api_folder, module_path] + ignore_paths
+ argv = ["-M", "--separate", "-o", api_folder, module_path, *ignore_paths]
apidoc.OPTIONS.append("ignore-module-all")
apidoc.main(argv)
diff --git a/markdown_it/common/normalize_url.py b/markdown_it/common/normalize_url.py
index a4ebbaae..92720b31 100644
--- a/markdown_it/common/normalize_url.py
+++ b/markdown_it/common/normalize_url.py
@@ -1,6 +1,7 @@
from __future__ import annotations
from collections.abc import Callable
+from contextlib import suppress
import re
from urllib.parse import quote, unquote, urlparse, urlunparse # noqa: F401
@@ -21,18 +22,17 @@ def normalizeLink(url: str) -> str:
"""
parsed = mdurl.parse(url, slashes_denote_host=True)
- if parsed.hostname:
- # Encode hostnames in urls like:
- # `http://host/`, `https://host/`, `mailto:user@host`, `//host/`
- #
- # We don't encode unknown schemas, because it's likely that we encode
- # something we shouldn't (e.g. `skype:name` treated as `skype:host`)
- #
- if not parsed.protocol or parsed.protocol in RECODE_HOSTNAME_FOR:
- try:
- parsed = parsed._replace(hostname=_punycode.to_ascii(parsed.hostname))
- except Exception:
- pass
+ # Encode hostnames in urls like:
+ # `http://host/`, `https://host/`, `mailto:user@host`, `//host/`
+ #
+ # We don't encode unknown schemas, because it's likely that we encode
+ # something we shouldn't (e.g. `skype:name` treated as `skype:host`)
+ #
+ if parsed.hostname and (
+ not parsed.protocol or parsed.protocol in RECODE_HOSTNAME_FOR
+ ):
+ with suppress(Exception):
+ parsed = parsed._replace(hostname=_punycode.to_ascii(parsed.hostname))
return mdurl.encode(mdurl.format(parsed))
@@ -47,18 +47,17 @@ def normalizeLinkText(url: str) -> str:
"""
parsed = mdurl.parse(url, slashes_denote_host=True)
- if parsed.hostname:
- # Encode hostnames in urls like:
- # `http://host/`, `https://host/`, `mailto:user@host`, `//host/`
- #
- # We don't encode unknown schemas, because it's likely that we encode
- # something we shouldn't (e.g. `skype:name` treated as `skype:host`)
- #
- if not parsed.protocol or parsed.protocol in RECODE_HOSTNAME_FOR:
- try:
- parsed = parsed._replace(hostname=_punycode.to_unicode(parsed.hostname))
- except Exception:
- pass
+ # Encode hostnames in urls like:
+ # `http://host/`, `https://host/`, `mailto:user@host`, `//host/`
+ #
+ # We don't encode unknown schemas, because it's likely that we encode
+ # something we shouldn't (e.g. `skype:name` treated as `skype:host`)
+ #
+ if parsed.hostname and (
+ not parsed.protocol or parsed.protocol in RECODE_HOSTNAME_FOR
+ ):
+ with suppress(Exception):
+ parsed = parsed._replace(hostname=_punycode.to_unicode(parsed.hostname))
# add '%' to exclude list because of https://github.com/markdown-it/markdown-it/issues/720
return mdurl.decode(mdurl.format(parsed), mdurl.DECODE_DEFAULT_CHARS + "%")
diff --git a/markdown_it/main.py b/markdown_it/main.py
index acf8d079..243e1509 100644
--- a/markdown_it/main.py
+++ b/markdown_it/main.py
@@ -4,11 +4,11 @@
from contextlib import contextmanager
from typing import Any, Literal, overload
-from . import helpers, presets # noqa F401
-from .common import normalize_url, utils # noqa F401
-from .parser_block import ParserBlock # noqa F401
-from .parser_core import ParserCore # noqa F401
-from .parser_inline import ParserInline # noqa F401
+from . import helpers, presets
+from .common import normalize_url, utils
+from .parser_block import ParserBlock
+from .parser_core import ParserCore
+from .parser_inline import ParserInline
from .renderer import RendererHTML, RendererProtocol
from .rules_core.state_core import StateCore
from .token import Token
diff --git a/markdown_it/presets/__init__.py b/markdown_it/presets/__init__.py
index 22cf74cb..f1cb0507 100644
--- a/markdown_it/presets/__init__.py
+++ b/markdown_it/presets/__init__.py
@@ -6,7 +6,7 @@
js_default = default
-class gfm_like:
+class gfm_like: # noqa: N801
"""GitHub Flavoured Markdown (GFM) like.
This adds the linkify, table and strikethrough components to CommmonMark.
diff --git a/markdown_it/renderer.py b/markdown_it/renderer.py
index 4cddbc67..7fee9ffa 100644
--- a/markdown_it/renderer.py
+++ b/markdown_it/renderer.py
@@ -152,19 +152,18 @@ def renderToken(
if token.block:
needLf = True
- if token.nesting == 1:
- if idx + 1 < len(tokens):
- nextToken = tokens[idx + 1]
-
- if nextToken.type == "inline" or nextToken.hidden:
- # Block-level tag containing an inline tag.
- #
- needLf = False
-
- elif nextToken.nesting == -1 and nextToken.tag == token.tag:
- # Opening tag + closing tag of the same type. E.g. ``.
- #
- needLf = False
+ if token.nesting == 1 and (idx + 1 < len(tokens)):
+ nextToken = tokens[idx + 1]
+
+ if nextToken.type == "inline" or nextToken.hidden: # noqa: SIM114
+ # Block-level tag containing an inline tag.
+ #
+ needLf = False
+
+ elif nextToken.nesting == -1 and nextToken.tag == token.tag:
+ # Opening tag + closing tag of the same type. E.g. ``.
+ #
+ needLf = False
result += ">\n" if needLf else ">"
diff --git a/markdown_it/ruler.py b/markdown_it/ruler.py
index 421666cc..8ae32beb 100644
--- a/markdown_it/ruler.py
+++ b/markdown_it/ruler.py
@@ -30,7 +30,7 @@ class Ruler
class StateBase:
- srcCharCode: tuple[int, ...]
+ srcCharCode: tuple[int, ...] # noqa: N815
def __init__(self, src: str, md: MarkdownIt, env: EnvType):
self.src = src
diff --git a/markdown_it/rules_block/fence.py b/markdown_it/rules_block/fence.py
index b4b28979..2051b96b 100644
--- a/markdown_it/rules_block/fence.py
+++ b/markdown_it/rules_block/fence.py
@@ -38,9 +38,8 @@ def fence(state: StateBlock, startLine: int, endLine: int, silent: bool) -> bool
params = state.src[pos:maximum]
# /* ` */
- if marker == 0x60:
- if chr(marker) in params:
- return False
+ if marker == 0x60 and chr(marker) in params:
+ return False
# Since start is found, we can report success here in validation mode
if silent:
diff --git a/markdown_it/rules_block/list.py b/markdown_it/rules_block/list.py
index eaaccda5..f1cb089e 100644
--- a/markdown_it/rules_block/list.py
+++ b/markdown_it/rules_block/list.py
@@ -120,14 +120,17 @@ def list_block(state: StateBlock, startLine: int, endLine: int, silent: bool) ->
# limit conditions when list can interrupt
# a paragraph (validation mode only)
- if silent and state.parentType == "paragraph":
- # Next list item should still terminate previous list item
- #
- # This code can fail if plugins use blkIndent as well as lists,
- # but I hope the spec gets fixed long before that happens.
- #
- if state.tShift[startLine] >= state.blkIndent:
- isTerminatingParagraph = True
+ # Next list item should still terminate previous list item
+ #
+ # This code can fail if plugins use blkIndent as well as lists,
+ # but I hope the spec gets fixed long before that happens.
+ #
+ if (
+ silent
+ and state.parentType == "paragraph"
+ and state.tShift[startLine] >= state.blkIndent
+ ):
+ isTerminatingParagraph = True
# Detect list type and position after marker
posAfterMarker = skipOrderedListMarker(state, startLine)
@@ -149,9 +152,11 @@ def list_block(state: StateBlock, startLine: int, endLine: int, silent: bool) ->
# If we're starting a new unordered list right after
# a paragraph, first line should not be empty.
- if isTerminatingParagraph:
- if state.skipSpaces(posAfterMarker) >= state.eMarks[startLine]:
- return False
+ if (
+ isTerminatingParagraph
+ and state.skipSpaces(posAfterMarker) >= state.eMarks[startLine]
+ ):
+ return False
# We should terminate list on style change. Remember first one to compare.
markerCharCode = state.srcCharCode[posAfterMarker - 1]
@@ -209,11 +214,8 @@ def list_block(state: StateBlock, startLine: int, endLine: int, silent: bool) ->
contentStart = pos
- if contentStart >= maximum:
- # trimming space in "- \n 3" case, indent is 1 here
- indentAfterMarker = 1
- else:
- indentAfterMarker = offset - initial
+ # trimming space in "- \n 3" case, indent is 1 here
+ indentAfterMarker = 1 if contentStart >= maximum else offset - initial
# If we have more than 4 spaces, the indent is 1
# (the rest is just indented code block)
diff --git a/markdown_it/rules_block/reference.py b/markdown_it/rules_block/reference.py
index 48f12721..92f0918c 100644
--- a/markdown_it/rules_block/reference.py
+++ b/markdown_it/rules_block/reference.py
@@ -153,18 +153,17 @@ def reference(state: StateBlock, startLine: int, _endLine: int, silent: bool) ->
break
pos += 1
- if pos < maximum and charCodeAt(string, pos) != 0x0A:
- if title:
- # garbage at the end of the line after title,
- # but it could still be a valid reference if we roll back
- title = ""
- pos = destEndPos
- lines = destEndLineNo
- while pos < maximum:
- ch = charCodeAt(string, pos)
- if not isSpace(ch):
- break
- pos += 1
+ if pos < maximum and charCodeAt(string, pos) != 0x0A and title:
+ # garbage at the end of the line after title,
+ # but it could still be a valid reference if we roll back
+ title = ""
+ pos = destEndPos
+ lines = destEndLineNo
+ while pos < maximum:
+ ch = charCodeAt(string, pos)
+ if not isSpace(ch):
+ break
+ pos += 1
if pos < maximum and charCodeAt(string, pos) != 0x0A:
# garbage at the end of the line
diff --git a/markdown_it/rules_block/state_block.py b/markdown_it/rules_block/state_block.py
index 02f8dc9c..ee77f097 100644
--- a/markdown_it/rules_block/state_block.py
+++ b/markdown_it/rules_block/state_block.py
@@ -202,10 +202,11 @@ def getLines(self, begin: int, end: int, indent: int, keepLastLF: bool) -> str:
while line < end:
lineIndent = 0
lineStart = first = self.bMarks[line]
- if line + 1 < end or keepLastLF:
- last = self.eMarks[line] + 1
- else:
- last = self.eMarks[line]
+ last = (
+ self.eMarks[line] + 1
+ if line + 1 < end or keepLastLF
+ else self.eMarks[line]
+ )
while (first < last) and (lineIndent < indent):
ch = self.srcCharCode[first]
diff --git a/markdown_it/rules_core/replacements.py b/markdown_it/rules_core/replacements.py
index e5d81c7a..0b6e86af 100644
--- a/markdown_it/rules_core/replacements.py
+++ b/markdown_it/rules_core/replacements.py
@@ -78,29 +78,30 @@ def replace_rare(inlineTokens: list[Token]) -> None:
inside_autolink = 0
for token in inlineTokens:
- if token.type == "text" and not inside_autolink:
- if RARE_RE.search(token.content):
- # +- -> ±
- token.content = PLUS_MINUS_RE.sub("±", token.content)
-
- # .., ..., ....... -> …
- token.content = ELLIPSIS_RE.sub("…", token.content)
-
- # but ?..... & !..... -> ?.. & !..
- token.content = ELLIPSIS_QUESTION_EXCLAMATION_RE.sub(
- "\\1..", token.content
- )
- token.content = QUESTION_EXCLAMATION_RE.sub("\\1\\1\\1", token.content)
-
- # ,, ,,, ,,,, -> ,
- token.content = COMMA_RE.sub(",", token.content)
-
- # em-dash
- token.content = EM_DASH_RE.sub("\\1\u2014", token.content)
-
- # en-dash
- token.content = EN_DASH_RE.sub("\\1\u2013", token.content)
- token.content = EN_DASH_INDENT_RE.sub("\\1\u2013", token.content)
+ if (
+ token.type == "text"
+ and (not inside_autolink)
+ and RARE_RE.search(token.content)
+ ):
+ # +- -> ±
+ token.content = PLUS_MINUS_RE.sub("±", token.content)
+
+ # .., ..., ....... -> …
+ token.content = ELLIPSIS_RE.sub("…", token.content)
+
+ # but ?..... & !..... -> ?.. & !..
+ token.content = ELLIPSIS_QUESTION_EXCLAMATION_RE.sub("\\1..", token.content)
+ token.content = QUESTION_EXCLAMATION_RE.sub("\\1\\1\\1", token.content)
+
+ # ,, ,,, ,,,, -> ,
+ token.content = COMMA_RE.sub(",", token.content)
+
+ # em-dash
+ token.content = EM_DASH_RE.sub("\\1\u2014", token.content)
+
+ # en-dash
+ token.content = EN_DASH_RE.sub("\\1\u2013", token.content)
+ token.content = EN_DASH_INDENT_RE.sub("\\1\u2013", token.content)
if token.type == "link_open" and token.info == "auto":
inside_autolink -= 1
diff --git a/markdown_it/rules_core/smartquotes.py b/markdown_it/rules_core/smartquotes.py
index b11a5739..b4284493 100644
--- a/markdown_it/rules_core/smartquotes.py
+++ b/markdown_it/rules_core/smartquotes.py
@@ -100,19 +100,17 @@ def process_inlines(tokens: list[Token], state: StateCore) -> None:
isLastWhiteSpace = isWhiteSpace(lastChar)
isNextWhiteSpace = isWhiteSpace(nextChar)
- if isNextWhiteSpace:
+ if isNextWhiteSpace: # noqa: SIM114
+ canOpen = False
+ elif isNextPunctChar and not (isLastWhiteSpace or isLastPunctChar):
canOpen = False
- elif isNextPunctChar:
- if not (isLastWhiteSpace or isLastPunctChar):
- canOpen = False
- if isLastWhiteSpace:
+ if isLastWhiteSpace: # noqa: SIM114
+ canClose = False
+ elif isLastPunctChar and not (isNextWhiteSpace or isNextPunctChar):
canClose = False
- elif isLastPunctChar:
- if not (isNextWhiteSpace or isNextPunctChar):
- canClose = False
- if nextChar == 0x22 and t.group(0) == '"': # 0x22: "
+ if nextChar == 0x22 and t.group(0) == '"': # 0x22: " # noqa: SIM102
if lastChar >= 0x30 and lastChar <= 0x39: # 0x30: 0, 0x39: 9
# special case: 1"" - count first quote as an inch
canClose = canOpen = False
diff --git a/markdown_it/rules_inline/balance_pairs.py b/markdown_it/rules_inline/balance_pairs.py
index ce0a0884..6125de71 100644
--- a/markdown_it/rules_inline/balance_pairs.py
+++ b/markdown_it/rules_inline/balance_pairs.py
@@ -60,10 +60,12 @@ def processDelimiters(state: StateInline, delimiters: list[Delimiter]) -> None:
# closing delimiters must not be a multiple of 3 unless both lengths
# are multiples of 3.
#
- if opener.close or closer.open:
- if (opener.length + closer.length) % 3 == 0:
- if opener.length % 3 != 0 or closer.length % 3 != 0:
- isOddMatch = True
+ if (
+ (opener.close or closer.open)
+ and ((opener.length + closer.length) % 3 == 0)
+ and (opener.length % 3 != 0 or closer.length % 3 != 0)
+ ):
+ isOddMatch = True
if not isOddMatch:
# If previous delimiter cannot be an opener, we can safely skip
diff --git a/markdown_it/rules_inline/entity.py b/markdown_it/rules_inline/entity.py
index 9c4c6a0e..1e5d0ea0 100644
--- a/markdown_it/rules_inline/entity.py
+++ b/markdown_it/rules_inline/entity.py
@@ -40,12 +40,11 @@ def entity(state: StateInline, silent: bool) -> bool:
else:
match = NAMED_RE.search(state.src[pos:])
- if match:
- if match.group(1) in entities:
- if not silent:
- state.pending += entities[match.group(1)]
- state.pos += len(match.group(0))
- return True
+ if match and match.group(1) in entities:
+ if not silent:
+ state.pending += entities[match.group(1)]
+ state.pos += len(match.group(0))
+ return True
if not silent:
state.pending += "&"
diff --git a/markdown_it/rules_inline/state_inline.py b/markdown_it/rules_inline/state_inline.py
index 7c1cb1f3..12e1d934 100644
--- a/markdown_it/rules_inline/state_inline.py
+++ b/markdown_it/rules_inline/state_inline.py
@@ -131,8 +131,6 @@ def scanDelims(self, start: int, canSplitWord: bool) -> Scanned:
"""
pos = start
- left_flanking = True
- right_flanking = True
maximum = self.posMax
marker = self.srcCharCode[start]
@@ -153,17 +151,14 @@ def scanDelims(self, start: int, canSplitWord: bool) -> Scanned:
isLastWhiteSpace = isWhiteSpace(lastChar)
isNextWhiteSpace = isWhiteSpace(nextChar)
- if isNextWhiteSpace:
- left_flanking = False
- elif isNextPunctChar:
- if not (isLastWhiteSpace or isLastPunctChar):
- left_flanking = False
-
- if isLastWhiteSpace:
- right_flanking = False
- elif isLastPunctChar:
- if not (isNextWhiteSpace or isNextPunctChar):
- right_flanking = False
+ left_flanking = not (
+ isNextWhiteSpace
+ or (isNextPunctChar and not (isLastWhiteSpace or isLastPunctChar))
+ )
+ right_flanking = not (
+ isLastWhiteSpace
+ or (isLastPunctChar and not (isNextWhiteSpace or isNextPunctChar))
+ )
if not canSplitWord:
can_open = left_flanking and ((not right_flanking) or isLastPunctChar)
diff --git a/markdown_it/token.py b/markdown_it/token.py
index e3f6c9b9..90008b72 100644
--- a/markdown_it/token.py
+++ b/markdown_it/token.py
@@ -80,7 +80,7 @@ def __post_init__(self) -> None:
self.attrs = convert_attrs(self.attrs)
def attrIndex(self, name: str) -> int:
- warnings.warn(
+ warnings.warn( # noqa: B028
"Token.attrIndex should not be used, since Token.attrs is a dictionary",
UserWarning,
)
diff --git a/pyproject.toml b/pyproject.toml
index acf2a288..22b220c8 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -83,6 +83,11 @@ exclude = [
profile = "black"
force_sort_within_sections = true
+[tool.ruff]
+line-length = 100
+extend-select = ["B0", "C4", "ICN", "ISC", "N", "RUF", "SIM"]
+extend-ignore = ["ISC003", "N802", "N803", "N806", "N816", "RUF003"]
+
[tool.mypy]
show_error_codes = true
warn_unused_ignores = true
diff --git a/tests/test_cli.py b/tests/test_cli.py
index c38e24fd..ed8d8205 100644
--- a/tests/test_cli.py
+++ b/tests/test_cli.py
@@ -37,7 +37,6 @@ def test_interactive():
def mock_input(prompt):
raise KeyboardInterrupt
- with patch("builtins.print") as patched:
- with patch("builtins.input", mock_input):
- parse.interactive()
+ with patch("builtins.print") as patched, patch("builtins.input", mock_input):
+ parse.interactive()
patched.assert_called()
diff --git a/tox.ini b/tox.ini
index 251e18df..59ea5f9e 100644
--- a/tox.ini
+++ b/tox.ini
@@ -66,7 +66,3 @@ description = run fuzzer on testcase file
deps = atheris
commands_pre = python scripts/build_fuzzers.py {envdir}/oss-fuzz
commands = python {envdir}/oss-fuzz/infra/helper.py reproduce markdown-it-py fuzz_markdown {posargs:testcase}
-
-[flake8]
-max-line-length = 100
-extend-ignore = E203