Skip to content
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

Add definition for "STR" #124

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add definition for "STR" #124

wants to merge 1 commit into from

Conversation

lucacasonato
Copy link
Member

No description provided.

@ljharb
Copy link
Member

ljharb commented Nov 29, 2022

O.o I’ve never seen this initialism used, where did this come up?

@ljharb
Copy link
Member

ljharb commented Nov 29, 2022

I now see, via matrix, that it was mentioned in one of the presentations.

It's fine to add, but I thought the point of this file is to document commonly used abbreviations/initialisms/terminology, not just to add every shorthand that ever comes up.

@ctcpip
Copy link
Member

ctcpip commented Nov 29, 2022

It's fine to add

Maybe not. While the bar needn't be high for including new terms, there should be a reasonable expectation that the term or acronym has some demonstrated substantial usage, at least in TC39, if not in general. It appears this doesn't pass muster.

@mpcsh
Copy link
Member

mpcsh commented Jan 4, 2023

+1 to Jordan's and Chris's points. I would be mildly against adding this term — I feel like the terminology doc has some kind of "blessing" effect, and I wouldn't want to see this initialism gain traction. It's uncommon (at least in my experience), and the term it replaces ("steps to reproduce") doesn't have a consistent, exact phrasing in common use. Furthermore, in my experience, "repro" is already in common use as a shorthand for this concept.

@syg
Copy link
Contributor

syg commented Jan 4, 2023

STR is pretty commonly used in implementations' issue trackers, but I also feel like it doesn't come up for committee.

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.

5 participants