Skip to content

IJPL-92400 Support GitHub alert syntax#189

Merged
Ilia Permiashkin (p3rmiashkin) merged 2 commits into
JetBrains:masterfrom
p3rmiashkin:ilia.permiashkin/alerts
Jun 3, 2026
Merged

IJPL-92400 Support GitHub alert syntax#189
Ilia Permiashkin (p3rmiashkin) merged 2 commits into
JetBrains:masterfrom
p3rmiashkin:ilia.permiashkin/alerts

Conversation

@p3rmiashkin

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tests fail

@p3rmiashkin

Copy link
Copy Markdown
Contributor Author

All tests seem to pass now

@donnerpeter

Copy link
Copy Markdown
Member

What was wrong with regexes?

@p3rmiashkin

Ilia Permiashkin (p3rmiashkin) commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

I don't think there was anything wrong with them. As Ivan said, this repo is based on the Kotlin Multiplatform framework, so JS wasn't happy about them for some reason, so I switched to simple string matching

@p3rmiashkin

Copy link
Copy Markdown
Contributor Author

Claude says the JS regex engine disguises brackets differently (compared to the Java engine) and it wasn't happy about \[ ] combination

@p3rmiashkin Ilia Permiashkin (p3rmiashkin) merged commit 037769f into JetBrains:master Jun 3, 2026
3 checks passed
@p3rmiashkin Ilia Permiashkin (p3rmiashkin) deleted the ilia.permiashkin/alerts branch June 3, 2026 18:14
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