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 apply refactoring returning -1 positions in ranges #502

Merged
merged 1 commit into from Jun 24, 2022
Merged

Conversation

rchl
Copy link
Member

@rchl rchl commented Jun 24, 2022

Typescript appears to be sending 0, 0 ranges when providing edit for
creating new file, even though the positions are supposed to be 1-based.

Fixes #501

Typescript appears to be sending 0, 0 ranges when providing edit for
creating new file, even though the positions are supposed to be 1-based.

Fixes #501
@rchl rchl merged commit b1f2b80 into master Jun 24, 2022
@rchl rchl deleted the fix/apply-edit branch June 24, 2022 22:33
rchl added a commit that referenced this pull request Jul 23, 2022
* master:
  chore(deps): add renovate configuration
  chore(lint): enable "no-constant-binary-expression" rule
  chore(deps): bump commander from 9.3.0 to 9.4.0 (#527)
  chore(deps-dev): bump @typescript-eslint/eslint-plugin from 5.28.0 to 5.30.7 (#521)
  chore(deps-dev): bump @typescript-eslint/parser from 5.30.6 to 5.30.7 (#526)
  chore(deps-dev): bump concurrently from 7.2.2 to 7.3.0 (#523)
  chore(deps-dev): bump eslint from 8.18.0 to 8.20.0 (#522)
  chore(deps-dev): bump @typescript-eslint/parser from 5.27.1 to 5.30.6 (#515)
  chore(deps-dev): bump ts-node from 10.8.1 to 10.9.1 (#518)
  chore(master): release 0.11.2 (#503)
  chore(deps-dev): bump eslint from 8.17.0 to 8.18.0 (#498)
  chore(deps-dev): bump @typescript-eslint/eslint-plugin from 5.27.1 to 5.28.0 (#499)
  chore(deps-dev): bump concurrently from 7.2.1 to 7.2.2 (#500)
  fix: apply refactoring returns -1 positions in ranges (#502)
  chore(master): release 0.11.1 (#494)
  fix: completion for strings with trigger character (#492)
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.

Refactor/code action "Move to a new file" has stopped working
1 participant