-
Notifications
You must be signed in to change notification settings - Fork 13
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
adapt instructions.md for CNTI #92
Conversation
instructions.md
Outdated
| `name` | Name of the product being certified. | | ||
| `version` | The version of the product being certified. | | ||
| `website_url` | URL to the product information website | | ||
| `repo_url` | If your product is open source, this field is necessary to point to the primary GitHub repo containing the source. It's OK if this is a mirror. OPTIONAL | | ||
| `documentation_url` | URL to the product documentation. OPTIONAL | | ||
| `product_logo_url` | URL to the product's logo, (must be in SVG format -- not a PNG -- and include the product name). If not supplied, we'll use your company logo. Please see logo [guidelines](https://github.com/cncf/landscape#logos). | | ||
| `product_logo_url` | URL to the product's logo, (must be in SVG format -- not a PNG -- and include the product name). If not supplied, we'll use your company logo. (TODO: LFN logo guidelines not found) Please see logo [guidelines](https://github.com/cncf/landscape#logos). | |
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.
@martin-mat @lixuna @Smitholi67 Maybe these guidelines can be copied over to the cnti certification repo or the cnti main repo. Either way that can be a new pull request
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.
lgtm
Refs lfn-cnti#74 Signed-off-by: Martin Matyas <martin.matyas@tietoevry.com>
ddeac28
to
f69068a
Compare
Certified product logo guidelines left to CNCF and not intended to adapt it in this PR. Created a new issue to address it subsequemntly (#94). Ready to review/merge. |
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.
lgtm
Refs #74
Pre-submission checklist:
Please check each of these after submitting your pull request: