Skip to content

fix: mark successive dots as noise and invalidate jinja syntax abbreviations #80

Merged
rzhao271 merged 7 commits into
microsoft:mainfrom
iifawzi:fix/jinga-successive--abbreviations
Jun 7, 2023
Merged

fix: mark successive dots as noise and invalidate jinja syntax abbreviations #80
rzhao271 merged 7 commits into
microsoft:mainfrom
iifawzi:fix/jinga-successive--abbreviations

Conversation

@iifawzi

@iifawzi iifawzi commented May 31, 2023

Copy link
Copy Markdown
Contributor

Hi, this PR should fix microsoft/vscode#179422
let me know if it could be done in any better way

iifawzi added 3 commits May 31, 2023 20:46
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
…date

Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
@iifawzi iifawzi force-pushed the fix/jinga-successive--abbreviations branch from 39c430e to 3bcd26e Compare May 31, 2023 21:44
@iifawzi

iifawzi commented Jun 7, 2023

Copy link
Copy Markdown
Contributor Author

Hi @rzhao271, would appreciate if you can take a look whenever you have a time

@rzhao271 rzhao271 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I have added some comments

Comment thread src/emmetHelper.ts Outdated
Comment thread src/emmetHelper.ts Outdated
Comment thread src/test/expand.test.ts
iifawzi added 3 commits June 7, 2023 20:04
… successive dots

Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
@iifawzi

iifawzi commented Jun 7, 2023

Copy link
Copy Markdown
Contributor Author

Pushed the changes https://github.com/iifawzi/vscode-emmet-helper/tree/fix/jinga-successive--abbreviations, but for some reason they didn't reflect here. That's weird, I've never encountered this.

EDIT: ok it seems like there's an outage, https://www.githubstatus.com/

Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
@iifawzi iifawzi force-pushed the fix/jinga-successive--abbreviations branch from be23333 to cb173b7 Compare June 7, 2023 17:56
@rzhao271 rzhao271 self-assigned this Jun 7, 2023
@rzhao271 rzhao271 added this to the June 2023 milestone Jun 7, 2023
@rzhao271 rzhao271 merged commit 64ecc72 into microsoft:main Jun 7, 2023
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.

Emmet suggestions produces incorrect Jinja syntax

3 participants