Skip to content

Remove setting python.formatting.formatOnSave in favor of the vs code setting #312

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 28 commits into from
Nov 28, 2017

Conversation

DonJayamanne
Copy link

@DonJayamanne DonJayamanne commented Nov 28, 2017

Fixes #285

  • This setting is deprecated in favour of the standard vs code setting to enable the feature of formattig on save.
  • Also took the liberty of refactoring some files (reduces code review for re-engineering of execution engine).
  • Removed unused file 'windowsService.ts'

octref and others added 27 commits November 3, 2017 13:11
* 'master' of https://github.com/Microsoft/vscode-python:
  Fixes #56 list all environments (#219)
  Fixes #57 Disable activation on debugging (#220)
  Fixes #26 Do not run linters when linters are disabled (#222)
* upstream/master:
  Fix typo in README.md (#252)
  Disable linter without workspaces (#241)
* upstream/master:
  Fix feedback service (#246)
  Fix django context initializer (#248)
  disable generation of tags file upon extension load (#264)
* upstream/master:
  Resolve pythonPath before comparing it to shebang (#273)
* upstream/master:
  Fixes #22 to Detect anaconda from known locations  (#221)
  Use workspaceFolder token instead of workspaceRoot (#267)
  Fix registry lookup response (#224)
  Fix issues when running without debugging and debugged code terminates (#249)
* upstream/master:
  Fix debugging tests (#304)
* upstream/master:
  Remove jupyter functionality in favor of Jupyter extension (#302)
  Drop Python 2 URLs (#307)
* master:
  Remove jupyter functionality in favor of Jupyter extension (#302)
  Drop Python 2 URLs (#307)
@DonJayamanne DonJayamanne added this to the December 2017 milestone Nov 28, 2017
const setting = deprecatedInfo.setting!;
let notify = false;
if (Array.isArray(workspace.workspaceFolders) && workspace.workspaceFolders.length > 0) {
workspace.workspaceFolders.forEach(wkspaceFolder => {
Copy link
Member

Choose a reason for hiding this comment

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

Why abbreviate wkspaceFolder for just two letters?

Copy link
Author

Choose a reason for hiding this comment

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

oh dam, forgot workspaceFolder isn't a reserved work. Its a habit, cuz workspace is a namespace and i had abbreviated in other places to wkspace.

Copy link
Author

Choose a reason for hiding this comment

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

Fixed

@DonJayamanne DonJayamanne merged commit c1edd58 into microsoft:master Nov 28, 2017
@DonJayamanne DonJayamanne deleted the removeFormatOnSave branch December 12, 2017 21:22
DonJayamanne added a commit that referenced this pull request Dec 14, 2017
* upstream/master:
  Remove setting python.formatting.formatOnSave in favor of the vs code setting (#312)
@lock lock bot locked as resolved and limited conversation to collaborators Jul 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

python.formatting.formatOnSave false seems to have no effect
3 participants