-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Introduce %$
option to add number of the test to its title
#14710
Introduce %$
option to add number of the test to its title
#14710
Conversation
✅ Deploy Preview for jestjs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
@mattphillips thoughts? I like the feature, but I'm also unsure about using |
This PR is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days. |
@mattphillips would love your input here 😀 I'm inclined to land it 🙂 |
Hey @kemuridama thanks for the PR, the code looks good to me to be merged. I'm not 100% on |
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.
🚀
Thanks!
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
%#
option to add index of the test to its title has been introduced by #6414. Similarly to it, I want to add number of the test to its title.This PR proposes to add
%$
option to add number of the test to its title.I don't know that
%$
is suitable as a placeholder. So I'd appreciate it if you could consider that point.Test plan
Tests have been added.