-
Notifications
You must be signed in to change notification settings - Fork 59
AI #1522: Renaming recency last update date to recency last updated #1534
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
Conversation
gparker-at-sf
left a comment
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.
Following TF3 discussion -- LGTM
|
As part of PR #1533 (beyond 1.3 Consistency Review scope) we started working on Metadata Guidelines and Naming Convention. Naming Convention Principles
Here is an overview of our property suffixes for the so-called State Indicator and Timestamp properties:
* Asterisk indicates a required naming change. Based on this, the 1.3 Consistency Review scope offers two options: Option 1: MinimumEnsure at least alignment among the three newly introduced Timestamp properties:
Option 2: AlternativeFully align all three newly introduced Timestamp properties and one newly introduced State Indicator property with the assumed naming convention:
Given the current scope, I agree we proceed with Option 1 for the 1.3 consistency review. That said, I intend to advocate for the North Star naming convention in 1.4, including revisiting and renaming even these newly introduced properties to align with the finalized guidelines. 🙂 |
shawnalpay
left a comment
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.
See recent comments.
Co-authored-by: Shawn Alpay <[email protected]>
Co-authored-by: Irena Jurica <[email protected]>
shawnalpay
left a comment
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.
Recency.mdpp is being reworked as part of #1521, but I'm good with approving this one!
|
Approved by the group without objections or abstentions during the October 30 Members’ call. |
No description provided.