Skip to content

Conversation

@jchatfranklin
Copy link

@jchatfranklin jchatfranklin commented Sep 5, 2025

Added many items in the Comments section, as most were questions and I was uncertain how to indicate those in the actual .md file.

Description

Please provide a meaningful description of what this change will do, or is for.
Bonus points for including links to related issues, other PRs, or technical
references.

Note that by not including a description, you are asking reviewers to do extra
work to understand the context of this change, which may lead to your PR taking
much longer to review, or result in it not being reviewed at all.

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

Added many items in the Comments section, as most were questions and I was uncertain how to indicate those in the actual .md file.

Signed-off-by: jchatfranklin <125913195+jchatfranklin@users.noreply.github.com>
@jchatfranklin
Copy link
Author

Please see Comments section of the revised file for questions and general comments on the document.

@beryder beryder mentioned this pull request Sep 9, 2025
@beryder
Copy link
Contributor

beryder commented Sep 9, 2025

Reference Issue: #137
I suggest that you copy each of your general notes and specific questions as separate comments here - and then we can start threads to discuss and resolve each one as needed


You can use the `Issuer CLI` to verify a published badge any published badge, not just those that you issued yourself.
This allows others to verify the Agent and MCP badges you publish.
You can use the `Issuer CLI` to verify any published badge -- not just those you issued yourself.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
You can use the `Issuer CLI` to verify any published badge -- not just those you issued yourself.
You can use the `Issuer CLI` to verify any published badge - not just those you issued yourself.

To run these steps, you must either:

- [Docker Desktop](https://docs.docker.com/get-docker/), or have both: [Docker Engine v27 or higher](https://docs.docker.com/engine/install/) and [Docker Compose v2.35 or higher](https://docs.docker.com/compose/install/)
- install [Docker Desktop](https://docs.docker.com/get-docker/), or
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capitalize the list items


- [Docker Desktop](https://docs.docker.com/get-docker/), or have both: [Docker Engine v27 or higher](https://docs.docker.com/engine/install/) and [Docker Compose v2.35 or higher](https://docs.docker.com/compose/install/)
- install [Docker Desktop](https://docs.docker.com/get-docker/), or
- have both [Docker Engine v27 or higher](https://docs.docker.com/engine/install/) and [Docker Compose v2.35 or higher](https://docs.docker.com/compose/install/)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here as well

Co-authored-by: Ben Ryder <142427115+beryder@users.noreply.github.com>
Signed-off-by: jchatfranklin <125913195+jchatfranklin@users.noreply.github.com>
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.

3 participants