-
Notifications
You must be signed in to change notification settings - Fork 70
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
[WG 2018-09-11] Relax specification for buildpack and stack IDs #7
Conversation
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.
Some typos
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've added some wording suggestions, but nothing that would stop me from approving.
buildpack.md
Outdated
The buildpack ID: | ||
- MUST only contain numbers, letters, and the charactors `.`, `/`, and `-`. | ||
- MUST NOT be `config` or `app`. | ||
- MUST NOT be identical to any other buildpack ID when letters in each ID are converted to lower case. |
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.
Should this be described as "MUST NOT be identical to any other buildpack ID when using a case-insensitive comparison"?
buildpack.md
Outdated
|
||
The stack ID: | ||
- MUST only contain numbers, letters, and the charactors `.`, `/`, and `-`. | ||
- MUST NOT be identical to any other stack ID when letters in each ID are converted to lower case. |
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.
Should this be described as "MUST NOT be identical to any other stack ID when using a case-insensitive comparison"?
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.
👍
@sclevine sorry so slow |
No description provided.