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

Dropped THtmlEscapedString #7285

Merged
merged 2 commits into from Jan 3, 2022
Merged

Conversation

weirdan
Copy link
Collaborator

@weirdan weirdan commented Jan 3, 2022

No description provided.

@weirdan weirdan added the release:removed The PR will be included in 'Removed' section of the release notes label Jan 3, 2022
@weirdan weirdan added this to the Psalm 5 milestone Jan 3, 2022
@weirdan weirdan requested a review from orklah January 3, 2022 20:50
@weirdan weirdan merged commit c2c3247 into vimeo:master Jan 3, 2022
@weirdan weirdan deleted the drop-html-escaped-string branch January 3, 2022 20:54
@orklah
Copy link
Collaborator

orklah commented Jan 3, 2022

@muglug do you know if TSqlSelectString is used by anyone? Does it makes sense to keep it?

@muglug
Copy link
Collaborator

muglug commented Jan 3, 2022

It doesn’t — ditch it!

@weirdan
Copy link
Collaborator Author

weirdan commented Jan 4, 2022

Oh, it doesn't exist anymore, only its token still remains. I'll drop it.

image

weirdan added a commit to weirdan/psalm that referenced this pull request Jan 4, 2022
@weirdan weirdan mentioned this pull request Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:removed The PR will be included in 'Removed' section of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants