Skip to content

Update entity definition (Issue #463) #504

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

Merged
merged 2 commits into from
Apr 6, 2019
Merged

Conversation

David-Chadwick
Copy link
Contributor

No description provided.

@@ -55,7 +55,7 @@
<dt><dfn data-lt="entities|entity's">entity</dfn></dt>
<dd>
A thing with distinct and independent existence, such as a person,
organization, concept, or device. An entity can perform one or more roles in
organization, or device that performs one or more roles in
Copy link
Member

Choose a reason for hiding this comment

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

This change blows up a number of sentences in the specification. For example:

A subject is an entity about which claims can be made.

So, the change of the definition creates a cascading set of changes to the specification that are not going to be able to reconcile. :(

Copy link
Member

Choose a reason for hiding this comment

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

Okay, maybe I was freaking out prematurely. I'd rather not make this change, but it looks like we have consensus in the issue among multiple people that regularly disagree about terminology, so that's something I don't want to get in the way of. :)

Merging, I'll make other non-normative changes to align other spec text to this new definition.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, we used to say "An entity may perform"... and it was changed to "can" because people were concerned about the normative-ness of the statement, which subtly changed what the definition was saying. Anyway, merging. Editorial, I'll do the re-analysis of how the definition is used in the entire spec and make sure there are no normative changes as a result.

@@ -55,7 +55,7 @@
<dt><dfn data-lt="entities|entity's">entity</dfn></dt>
<dd>
A thing with distinct and independent existence, such as a person,
organization, concept, or device. An entity can perform one or more roles in
organization, or device that performs one or more roles in
Copy link
Member

Choose a reason for hiding this comment

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

Yeah, we used to say "An entity may perform"... and it was changed to "can" because people were concerned about the normative-ness of the statement, which subtly changed what the definition was saying. Anyway, merging. Editorial, I'll do the re-analysis of how the definition is used in the entire spec and make sure there are no normative changes as a result.

@msporny msporny merged commit d2fc070 into gh-pages Apr 6, 2019
@msporny msporny deleted the entity-definition branch April 24, 2019 04:09
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.

4 participants