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

Fix indentation problem after multi-line Python function definition #14748

Merged
merged 4 commits into from
Jul 7, 2023

Conversation

FoSuCloud
Copy link
Contributor

@FoSuCloud FoSuCloud commented Jun 24, 2023

References

Fixes #14668

Code changes

Use resolutions to limit the @codemirror/lang-python version
image

Execute "yarn install" and "jlpm build" to update dependencies
image

User-facing changes

nothing

Backwards-incompatible changes

nothing

@jupyterlab-probot
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@FoSuCloud
Copy link
Contributor Author

@meeseeksdev tag bug

@lumberbot-app lumberbot-app bot added the bug label Jun 24, 2023
Copy link
Member

@fcollonval fcollonval left a comment

Choose a reason for hiding this comment

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

thanks @FoSuCloud

Could you update the specification a bit differently (see below comment)?

GitHub tip: if you use special keywords like Fixes in front of the issue in the PR description, the issue will be linked with this PR easing documentation and repository management 😉

package.json Outdated Show resolved Hide resolved
@FoSuCloud FoSuCloud changed the title fix: indentation problem after multi-line Python function definition Fix indentation problem after multi-line Python function definition Jun 26, 2023
Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

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

Thank you @FoSuCloud

@krassowski krassowski added this to the 4.0.x milestone Jul 7, 2023
@krassowski krassowski merged commit 2161e6c into jupyterlab:main Jul 7, 2023
@krassowski
Copy link
Member

@meeseeksdev please backport to 4.0.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Jul 7, 2023
krassowski pushed a commit that referenced this pull request Jul 8, 2023
…unction definition (#14801)

Co-authored-by: FoSuCloud <49218295+FoSuCloud@users.noreply.github.com>
smacke pushed a commit to smacke/jupyterlab that referenced this pull request Jul 8, 2023
…upyterlab#14748)

* fix: indentation problem after multi-line Python function definition

* fix: Remove duplicate yarn dependencies

* fix: update @codemirror/lang-python version

* fix: update @codemirror/lang-python version

---------

Co-authored-by: yelipei <yelipei@qianxin.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

indentation problem after multi-line Python function definition
3 participants