Skip to content

Tweaks to #709 #823

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

Conversation

pokey
Copy link
Member

@pokey pokey commented Jul 4, 2022

Checklist

@pokey pokey changed the base branch from main to textTheBasedItemScope July 4, 2022 14:09
@pokey pokey marked this pull request as ready for review July 4, 2022 14:56
@AndreasArvidsson AndreasArvidsson merged commit 6136843 into textTheBasedItemScope Jul 4, 2022
@AndreasArvidsson AndreasArvidsson deleted the pokey/text-based-item-tweaks branch July 4, 2022 16:17
AndreasArvidsson added a commit that referenced this pull request Jul 5, 2022
* Added text based item scope

* Cleaned up imports

* Use text based item as a fallback

* Updated tests

* Don't look for items inside strings

* More work on item stage

* updated  test

* Replaced take test with clear test

* updated item stage

* Typescript now uses text based item

* Use text based item for more languages

* Use text based item for more languages

* Cleanup

* Added more tests

* Added test

* Added support for escaped strings

* Added additional test

* Added more tests

* Light clean up and add test

* cleanup

* Include trailing delimiter in arguments

* Cleanup

* Cleanup

* Don't expand for weak item target

* Include pairs inside strings

* Cleanup

* p

* Added item support to python argument list

* Use new requireStrongContainment

* Cleanup

* Cleanup

* Added complex surrounding pair scope type

* Tweaks to #709 (#823)

* Tweaks to text-based item PR

* More tweaks and tests

* More doc string

Co-authored-by: Andreas Arvidsson <[email protected]>

* Revert "Added complex surrounding pair scope type"

This reverts commit 95acb52.

* Cleanup

* Create single union of items

* Excluded separator from domain

* Cleanup

* Updated tests

* Update src/processTargets/modifiers/ItemStage/tokenizeRange.ts

Co-authored-by: Pokey Rule <[email protected]>

* Update src/processTargets/modifiers/ItemStage/tokenizeRange.ts

Co-authored-by: Pokey Rule <[email protected]>

* Update src/languages/python.ts

Co-authored-by: Pokey Rule <[email protected]>

* Cleanup

* Tweaks

Co-authored-by: Pokey Rule <[email protected]>
Co-authored-by: Andreas Arvidsson <[email protected]>
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