Skip to content

Change runSelectionInActiveTerm effect to warning#351

Merged
renkun-ken merged 1 commit intoREditorSupport:masterfrom
andycraig:remove-send-active
Jun 6, 2020
Merged

Change runSelectionInActiveTerm effect to warning#351
renkun-ken merged 1 commit intoREditorSupport:masterfrom
andycraig:remove-send-active

Conversation

@andycraig
Copy link
Copy Markdown
Collaborator

Part of #306

What problem did you solve?

Replaces effect of R: Run Selection/Line in Active Terminal with warning message, in preparation for its eventual removal.

How can I check this pull request?

Use command R: Run Selection/Line in Active Terminal. Observe code is NOT sent to terminal. Observe warning message 'This command has been removed. Enable setting "Always Use Active Terminal".'

@renkun-ken
Copy link
Copy Markdown
Member

LGTM. Please feel free to merge if there's nothing to be added.

Comment thread src/extension.ts
@renkun-ken renkun-ken merged commit e5c8424 into REditorSupport:master Jun 6, 2020
@andycraig
Copy link
Copy Markdown
Collaborator Author

@renkun-ken Thank you for testing and reviewing! I will merge.

@andycraig
Copy link
Copy Markdown
Collaborator Author

@renkun-ken And thank you for merging! 😄

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