Skip to content

Conversation

@marcoscaceres
Copy link
Member

@marcoscaceres marcoscaceres commented Oct 14, 2025

This pull request improves the consistency and clarity of terminology related to credential management in the index.bs specification file. The main focus is on standardizing the use of the term "credential manager" by replacing plain text references with properly defined and linked terms, and by adding a clear definition of what a credential manager is.

Key terminology and documentation improvements:

  • Standardized all references to "credential manager" and "credential managers" by replacing plain text mentions with the defined term [=credential manager=] throughout the specification. [1] [2] [3] [4] [5]
  • Added a formal definition for credential manager, clarifying that it can be an application, hardware device, or service that stores, organizes, manages, and allows choosing credentials (e.g., digital wallets, password managers, passkey managers).

Preview | Diff

@marcoscaceres marcoscaceres requested a review from Copilot October 14, 2025 08:36
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request standardizes terminology for "credential manager" throughout the specification by adding a formal definition and replacing plain text references with properly linked terms. The changes improve consistency and clarity in the credential management specification.

  • Added a formal definition for "credential manager" explaining it as an application, hardware device, or service that manages credentials
  • Replaced all plain text mentions of "credential manager" and "credential managers" with linked references using [=credential manager=] syntax
  • Updated 6 locations throughout the specification to use the standardized terminology

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@nsatragno
Copy link
Member

Approved, thanks!

The PR description is really wordy, would it be possible to summarize it for the commit message?

@marcoscaceres marcoscaceres merged commit 1c2a8d4 into main Oct 22, 2025
1 of 2 checks passed
@marcoscaceres
Copy link
Member Author

Sorry about long blah, co-pilot goes off the rails sometimes. The summary was not in the commit message, so no worries.

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