-
Notifications
You must be signed in to change notification settings - Fork 240
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
Detect more test files #1297
Detect more test files #1297
Conversation
lib/shared/src/chat/prompts/utils.ts
Outdated
return ( | ||
fileNameWithoutExt.startsWith('test') || | ||
fileNameWithoutExt.endsWith('test') || | ||
fileNameWithoutExt.endsWith('tests') || | ||
fileNameWithoutExt.endsWith('spec') | ||
) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I will change this to a regex to avoid things like Protest
:D
['/path/to/testFile.java', false], | ||
['/path/to/testFile.js', false], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These two I broke in this PR since it doesn't seem to be from a popular test framework, but maybe I am overlooking something?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They were not! I wasn't doing any lang-specified lookup so anything that starts with test would have passed lol thanks for updating this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use AI tools to build AI tools 🧠
I asked GPT4 to give me some common examples and made the test pass. WDYT @abeatrix?
Test plan
Test green