Skip to content

docs: Clarify documentation for unique_keys, key, and identifier.#258

Merged
amc-corey-cox merged 1 commit into
linkml:mainfrom
gouttegd:fix-identifiers-spec
Jul 1, 2026
Merged

docs: Clarify documentation for unique_keys, key, and identifier.#258
amc-corey-cox merged 1 commit into
linkml:mainfrom
gouttegd:fix-identifiers-spec

Conversation

@gouttegd

Copy link
Copy Markdown
Contributor

This PR completes the fix for linkml/linkml#3329 on the side of the LinkML model.

It updates the inline documentation for the unique_keys, key, and identifier slot to better explain the differences between the three, and align it with the recent corresponding update to the LinkML documentation (linkml/linkml#3649).

Update the inline documentation for the `unique_keys`, `key`, and
`identifier` slot to better explain the differences between the three
types of "keys" and align it with the corresponding recent update to the
LinkML documentation (linkml/linkml#3649).

This completes the fix to linkml/linkml#3329.
@gouttegd gouttegd self-assigned this Jun 19, 2026
@gouttegd
gouttegd marked this pull request as ready for review June 19, 2026 10:03
@gouttegd
gouttegd requested a review from amc-corey-cox June 19, 2026 10:03
@gouttegd gouttegd added the documentation Improvements or additions to documentation label Jun 19, 2026

@dalito dalito left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for adding more information about keys/ids also here.

@amc-corey-cox amc-corey-cox left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to me.

@amc-corey-cox
amc-corey-cox merged commit d028ff3 into linkml:main Jul 1, 2026
8 checks passed
@gouttegd
gouttegd deleted the fix-identifiers-spec branch July 1, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants