Skip to content

Added full line scope - #3095

Merged
AndreasArvidsson merged 6 commits into
mainfrom
expandToFullLine
Nov 4, 2025
Merged

AndreasArvidsson merged 6 commits into
mainfrom
expandToFullLine

Conversation

@AndreasArvidsson

@AndreasArvidsson AndreasArvidsson commented Oct 31, 2025 •

Copy link
Copy Markdown
Member

Line scope that includes leading and trailing whitespace
"change full line"

@AndreasArvidsson
AndreasArvidsson requested a review from a team as a code owner October 31, 2025 10:29
pokey
pokey previously approved these changes Oct 31, 2025
@AndreasArvidsson

Copy link
Copy Markdown
Member Author

I'm started thinking that maybe I should convert this to scope instead? Might be useful to be able to do next expand? @pokey I think you had the idea of doing a scope full line and I think that might be a better solution.

@AndreasArvidsson
AndreasArvidsson marked this pull request as draft November 3, 2025 06:12
@pokey

pokey commented Nov 3, 2025

Copy link
Copy Markdown
Member

I'm started thinking that maybe I should convert this to scope instead? Might be useful to be able to do next expand? @pokey I think you had the idea of doing a scope full line and I think that might be a better solution.

Yeah that was my first instinct. I think either is fine

@AndreasArvidsson
AndreasArvidsson marked this pull request as ready for review November 4, 2025 09:04
@AndreasArvidsson

Copy link
Copy Markdown
Member Author

@pokey Now converted to a scope

@AndreasArvidsson AndreasArvidsson changed the title Added expandToFullLine modifier Added full line scope Nov 4, 2025
@AndreasArvidsson
AndreasArvidsson merged commit a855424 into main Nov 4, 2025
15 checks passed
@AndreasArvidsson
AndreasArvidsson deleted the expandToFullLine branch November 4, 2025 10:08
cursorless-bot pushed a commit that referenced this pull request Nov 11, 2025
Line scope that includes leading and trailing whitespace
`"change full line"`
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