Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Replace 'shape' with better terminology #611

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
merged 2 commits into from
Jan 21, 2024
Merged

Conversation

aphillips
Copy link
Member

@aphillips aphillips commented Jan 19, 2024

Addresses #590

Replaces the amorphous term "shape" in Formatting with more appropriate text. Generally I used a reformulation using some variation of the word "form".

In one case, I deleted the paragraph. The paragraph in question was non-normative and basically said the same thing as a normative paragraph just below it--that how formatting works is implementation-defined.


In fixing this, I corrected one typo.

I also transformed the eager/lazy note into an "IMPORTANT" (which also restores it's intended normativity)

Addresses #590

Replaces the amorphous term "shape" in Formatting with more appropriate text. Generally I used a reformulation using some variation of the word "form".

In one case, I deleted the paragraph. The paragraph in question was non-normative and basically said the same thing as a normative paragraph just below it--that how formatting works is implementation-defined.
@aphillips aphillips added editorial Issue is non-normative formatting Issue pertains to the formatting section of the spec LDML45 LDML45 Release (Tech Preview) labels Jan 19, 2024
The note about eager vs. lazy was intended to be normative. Putting it in a note makes it informative. I changed to "important" and fixed it to format correctly in markdown.
@aphillips aphillips added the normative Issue affects normative text in the specification label Jan 20, 2024
@aphillips aphillips requested a review from macchiati January 20, 2024 16:59
@aphillips
Copy link
Member Author

@macchiati I added a normative change, so canceling your previous approve.

@aphillips aphillips merged commit 59fc4b2 into main Jan 21, 2024
@aphillips aphillips deleted the aphillips-replace-shape branch January 21, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial Issue is non-normative formatting Issue pertains to the formatting section of the spec LDML45 LDML45 Release (Tech Preview) normative Issue affects normative text in the specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants