Skip to content

Add formatting of type parameters in class and function definitions#6161

Merged
zanieb merged 1 commit intomainfrom
zanie/695-format
Aug 2, 2023
Merged

Add formatting of type parameters in class and function definitions#6161
zanieb merged 1 commit intomainfrom
zanie/695-format

Conversation

@zanieb
Copy link
Member

@zanieb zanieb commented Jul 28, 2023 •
edited
Loading

Part of #5062
Closes #5931

Implements formatting of a sequence of type parameters in a dedicated struct for reuse by classes, functions, and type aliases (preparing for #5929). Adds formatting of type parameters in class and function definitions — previously, they were just elided.

@zanieb zanieb force-pushed the zanie/695-format branch from 48a4897 to 08f81c7 Compare July 28, 2023 19:25
@github-actions
Copy link
Contributor

github-actions bot commented Jul 28, 2023 •
edited
Loading

PR Check Results

Ecosystem

✅ ecosystem check detected no changes.

Benchmark

Linux

group                                      main                                   pr
-----                                      ----                                   --
formatter/large/dataset.py                 1.00      8.3±0.13ms     4.9 MB/sec    1.00      8.3±0.15ms     4.9 MB/sec
formatter/numpy/ctypeslib.py               1.02  1675.5±59.89µs     9.9 MB/sec    1.00  1648.6±31.97µs    10.1 MB/sec
formatter/numpy/globals.py                 1.00    186.7±6.27µs    15.8 MB/sec    1.01    188.1±6.58µs    15.7 MB/sec
formatter/pydantic/types.py                1.01      3.6±0.14ms     7.1 MB/sec    1.00      3.5±0.10ms     7.2 MB/sec
linter/all-rules/large/dataset.py          1.00     10.8±0.05ms     3.8 MB/sec    1.00     10.8±0.04ms     3.8 MB/sec
linter/all-rules/numpy/ctypeslib.py        1.00      2.8±0.03ms     6.0 MB/sec    1.00      2.8±0.02ms     6.0 MB/sec
linter/all-rules/numpy/globals.py          1.00    381.1±2.22µs     7.7 MB/sec    1.00    380.1±2.35µs     7.8 MB/sec
linter/all-rules/pydantic/types.py         1.00      4.9±0.03ms     5.2 MB/sec    1.03      5.0±0.13ms     5.1 MB/sec
linter/default-rules/large/dataset.py      1.00      5.7±0.04ms     7.1 MB/sec    1.01      5.8±0.05ms     7.1 MB/sec
linter/default-rules/numpy/ctypeslib.py    1.00   1196.6±4.60µs    13.9 MB/sec    1.00   1194.1±7.16µs    13.9 MB/sec
linter/default-rules/numpy/globals.py      1.00    132.6±1.71µs    22.3 MB/sec    1.00    132.4±1.29µs    22.3 MB/sec
linter/default-rules/pydantic/types.py     1.00      2.5±0.03ms    10.2 MB/sec    1.01      2.5±0.04ms    10.1 MB/sec

Windows

group                                      main                                    pr
-----                                      ----                                    --
formatter/large/dataset.py                 1.01     12.6±0.77ms     3.2 MB/sec     1.00     12.5±0.76ms     3.2 MB/sec
formatter/numpy/ctypeslib.py               1.04      2.5±0.19ms     6.6 MB/sec     1.00      2.4±0.19ms     6.9 MB/sec
formatter/numpy/globals.py                 1.00   261.8±20.98µs    11.3 MB/sec     1.01   265.4±25.19µs    11.1 MB/sec
formatter/pydantic/types.py                1.04      5.5±0.41ms     4.7 MB/sec     1.00      5.3±0.35ms     4.8 MB/sec
linter/all-rules/large/dataset.py          1.00     17.2±0.85ms     2.4 MB/sec     1.01     17.5±1.13ms     2.3 MB/sec
linter/all-rules/numpy/ctypeslib.py        1.02      4.6±0.34ms     3.6 MB/sec     1.00      4.5±0.38ms     3.7 MB/sec
linter/all-rules/numpy/globals.py          1.00   556.2±56.95µs     5.3 MB/sec     1.01   560.0±50.88µs     5.3 MB/sec
linter/all-rules/pydantic/types.py         1.00      7.8±0.45ms     3.3 MB/sec     1.02      7.9±0.47ms     3.2 MB/sec
linter/default-rules/large/dataset.py      1.00      9.5±0.51ms     4.3 MB/sec     1.00      9.5±0.64ms     4.3 MB/sec
linter/default-rules/numpy/ctypeslib.py    1.00  1926.2±129.22µs     8.6 MB/sec    1.00  1921.8±117.94µs     8.7 MB/sec
linter/default-rules/numpy/globals.py      1.00   216.3±23.63µs    13.6 MB/sec     1.05   226.6±27.81µs    13.0 MB/sec
linter/default-rules/pydantic/types.py     1.01      4.2±0.31ms     6.0 MB/sec     1.00      4.2±0.35ms     6.1 MB/sec

@charliermarsh
Copy link
Member

charliermarsh commented Jul 28, 2023

Ooooh yes

@zanieb zanieb marked this pull request as ready for review July 28, 2023 20:12
@zanieb zanieb mentioned this pull request Jul 28, 2023
Merged
@zanieb zanieb requested review from MichaReiser and konstin July 28, 2023 22:25
@konstin konstin mentioned this pull request Jul 31, 2023
Closed
72 tasks
MichaReiser
MichaReiser approved these changes Jul 31, 2023
View reviewed changes
Copy link
Member

@MichaReiser MichaReiser left a comment

Choose a reason for hiding this comment

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

Niiiice! This is looking good. I have two test cases that I recommend you adding and I think we can move some more logic into the clause formatting helper.

@konstin konstin added the formatter Related to the formatter label Jul 31, 2023
konstin
konstin approved these changes Jul 31, 2023
View reviewed changes
@zanieb zanieb mentioned this pull request Jul 31, 2023
Closed
@zanieb zanieb force-pushed the zanie/695-format branch 5 times, most recently from 655d055 to 4ac79ac Compare August 2, 2023 16:07
zanieb
zanieb commented Aug 2, 2023
View reviewed changes
crates/ruff_python_formatter/src/comments/placement.rs
}
}

/// Attach an enclosed end-of-line comment to a set of [`TypeParams`].
Copy link
Member Author

@zanieb zanieb Aug 2, 2023

Choose a reason for hiding this comment

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

As implemented in #6274

charliermarsh
charliermarsh approved these changes Aug 2, 2023
View reviewed changes
Copy link
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

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

Looks great.

@zanieb zanieb force-pushed the zanie/695-format branch from 4ac79ac to 3660033 Compare August 2, 2023 20:19
@zanieb zanieb enabled auto-merge (squash) August 2, 2023 20:21
@zanieb zanieb merged commit 1a60d1e into main Aug 2, 2023
@zanieb zanieb deleted the zanie/695-format branch August 2, 2023 20:29
zanieb added a commit that referenced this pull request Aug 2, 2023
@zanieb
Add formatting of type alias statements (#6162)
5b2e973 
Part of #5062 
Extends #6161
Closes #5929
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MichaReiser MichaReiser MichaReiser approved these changes

@konstin konstin konstin approved these changes

@charliermarsh charliermarsh charliermarsh approved these changes

Assignees

No one assigned

Labels

formatter Related to the formatter

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Format type parameters on functions and classes

4 participants

@zanieb @charliermarsh @MichaReiser @konstin