Skip to content
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

respect indent style preference #26

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

noneofyourbusiness1415252
Copy link
Contributor

@noneofyourbusiness1415252 noneofyourbusiness1415252 commented Nov 24, 2023

Make yapf respect indent style preference
tested @ https://replit.com/@UMARismyname/black#f.py

@noneofyourbusiness1415252 noneofyourbusiness1415252 marked this pull request as ready for review November 24, 2023 15:33
@noneofyourbusiness1415252 noneofyourbusiness1415252 changed the title efficient formatting -- compat respect indent style preference Nov 24, 2023
@noneofyourbusiness1415252 noneofyourbusiness1415252 marked this pull request as draft November 24, 2023 20:21
@noneofyourbusiness1415252 noneofyourbusiness1415252 force-pushed the patch-2-leg branch 3 times, most recently from 6ffa45a to 027d07d Compare November 24, 2023 20:28
@noneofyourbusiness1415252 noneofyourbusiness1415252 marked this pull request as ready for review November 24, 2023 20:29
@noneofyourbusiness1415252
Copy link
Contributor Author

ping me for a review once you do that

@jackyzha0

@jackyzha0
Copy link
Member

could you uncommit the package-lock.json? seems like you got some package bumps in there that are not relevant to your changes

packages/pyright/webpack.config.js Outdated Show resolved Hide resolved
@noneofyourbusiness1415252 noneofyourbusiness1415252 force-pushed the patch-2-leg branch 3 times, most recently from ea2cd3e to 73b90c2 Compare November 28, 2023 18:00
@jackyzha0
Copy link
Member

don't forget to run prettier! CI is failing on style check

This comment has been minimized.

returning to appreciating the formatting loading animation :(
Copy link
Contributor

Diff from mypy_primer, showing the effect of this PR on open source code:

pandas-stubs (https://github.com/pandas-dev/pandas-stubs): typechecking got 1.81x slower (14.6s -> 26.4s)
(Performance measurements are based on a single noisy sample)

@jackyzha0 jackyzha0 merged commit dc0cbef into replit:main Nov 29, 2023
9 checks passed
@noneofyourbusiness1415252 noneofyourbusiness1415252 deleted the patch-2-leg branch November 29, 2023 19:58
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.

2 participants