cf-convention / cf-conventions

AsciiDoc Source

http://cfconventions.org/cf-conventions/cf-conventions

Creative Commons Zero v1.0 Universal

85 stars 45 forks source link

Revert #478, remove inconsistent wording about special characters #548

Open Dave-Allured opened 2 weeks ago

Dave-Allured commented 2 weeks ago

Moderator

TBD

Associated pull request

549 Detailed Proposal

Revert PR #478. Remove this single line from section 2.3, Naming Conventions: "ASCII period (.) and ASCII hyphen (-) are also allowed in attribute names only."

This line was based on my misunderstanding of an earlier version of 2.3 paragraph 1. That paragraph was later clarified by PR #526, which now says that character restrictions are a recommendation, not a requirement. Therefore this new sentence is redundant and confusing.

JonathanGregory commented 1 week ago

Dear @Dave-Allured

Thanks for raising this issue. It is relevant to #528 and discussion 323, so I've flagged it in both. Since the proposal to add this sentence was accepted as an enhancement, with the required amount of support being expressed, I don't think we can reverse that decision as a defect i.e. if no-one objects. I think we need support for your proposal to remove the sentence as a new enhancement, so I'll change the label.

Best wishes

Jonathan

Dave-Allured commented 1 week ago

@JonathanGregory, thanks for switching the label. That is fine with me.