Skip to content

PSRL: Changed method names for consistency and introduced backwards compatible methods #7146

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 4 commits into from
Feb 24, 2021

Conversation

springcomp
Copy link
Contributor

PR Summary

This is related to PSRL #2002

PR Context

As part of this PR, PSRL’s BackwardDeleteLine has been changed so that it deletes text from the start of the current logical line to the cursor position, instead of deleting from the start of the input. This supports VI-mode’s d, 0 command.

A new method named BackwardDeleteInput has been introduced to replicate the behaviour of the BackwardDeleteLine method before this change.

For consistency, similar changes have been made to the following methods:

  • ForwardDeleteLine has been changed to apply to a logical line, and ForwardDeleteInput has been introduced to replicate the old behaviour.
  • BackwardKillLine has been changed to apply to a logical line, and BackwardKillInput has been introduced to replicate the old behaviour.

This PR updates the documentation for the changed methods and add documentation for the new methods.

Select the area of the Table of Contents containing the documents being changed.

Conceptual content

  • Overview and Install
  • Learning PowerShell
    • PowerShell 101
    • Deep dives
    • Remoting
  • Release notes (What's New)
  • Windows PowerShell
    • WMF, ISE, release notes, etc.
  • DSC articles
  • Community resources
  • Sample scripts
  • Gallery articles
  • Scripting and development
    • PSReadLine
    • Legacy SDK

Cmdlet reference & about_ topics

  • Preview content
  • Version 7.1 content
  • Version 7.0 content
  • Version 5.1 content

PR Checklist

  • I have read the contributors guide and followed the style and process guidelines
  • PR has a meaningful title
  • PR is targeted at the staging branch
  • All relevant versions updated
  • Includes content related to issues and PRs - see Closing issues using keywords.
  • This PR is ready to merge and is not Work in Progress
    • If the PR is work in progress, please add the prefix WIP: or [WIP] to the beginning of the
      title and remove the prefix when the PR is ready.

@opbld30
Copy link

opbld30 commented Jan 19, 2021

Docs Build status updates of commit 2079580:

✅ Validation status: passed

File Status Preview URL Details
reference/7.2/PSReadLine/About/about_PSReadLine.md ✅Succeeded View (powershell-7.2)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@sdwheeler sdwheeler added the hold-for-pr Waiting - for PR merge label Jan 19, 2021
@sdwheeler sdwheeler modified the milestones: 7.0.0, Future Jan 19, 2021
@chasewilson chasewilson removed the hold-for-pr Waiting - for PR merge label Feb 19, 2021
@sdwheeler sdwheeler modified the milestones: Future, 7.2.0 Feb 23, 2021
@sdwheeler sdwheeler merged commit 34a4987 into MicrosoftDocs:staging Feb 24, 2021
@springcomp springcomp deleted the PSRL_2002 branch February 24, 2021 22:32
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.

4 participants