Skip to content

Fix syntax test glob #27

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

Conversation

michaelblyons
Copy link
Contributor

@michaelblyons michaelblyons commented Feb 12, 2025

Also warn if there are any near-miss syntax file names.

Fixes #26

@michaelblyons michaelblyons force-pushed the workflow-sample/test-file-name branch from d979df0 to 1ad07f4 Compare February 12, 2025 15:40
@michaelblyons michaelblyons force-pushed the workflow-sample/test-file-name branch from 1ad07f4 to 2832a3b Compare February 12, 2025 16:10
@michaelblyons michaelblyons marked this pull request as ready for review February 12, 2025 16:17
@michaelblyons michaelblyons changed the title Update syntax file path in sample Fix syntax test glob Feb 12, 2025
@jrappen
Copy link

jrappen commented Feb 26, 2025

ST Build 4194+ now have this requirement, too.

The test binary for CI tests already had it.

@michaelblyons michaelblyons force-pushed the workflow-sample/test-file-name branch from b62e1b7 to abcbe3c Compare February 27, 2025 00:08
Copy link
Member

@FichteFoll FichteFoll left a comment

Choose a reason for hiding this comment

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

Thanks, good ideas.

@FichteFoll FichteFoll added this to the v2.5 milestone Apr 10, 2025
@FichteFoll FichteFoll merged commit ad44ed9 into SublimeText:master Apr 10, 2025
8 checks passed
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.

Syntax test file names
3 participants