Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[DESIGN] Add user stories / build-out of the expression attributes #792

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 4 commits into from
May 20, 2024

Conversation

aphillips
Copy link
Member

@aphillips aphillips commented May 14, 2024

Add user-stories and examples to the expression attributes design.

…bute design

Add user-stories and examples to the expression attributes design.
@aphillips aphillips requested a review from eemeli May 14, 2024 00:01
Copy link
Collaborator

@eemeli eemeli left a comment

Choose a reason for hiding this comment

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

Looks pretty good. The quoting of e.g. "locale" and "direction" in text looks a bit odd, and the code examples probably don't need > indentation.

A third user story could be added to account for the use case introduced in #772.

Remember to add this PR to the "Pull Requests" list.

@aphillips aphillips changed the title [DESIGN] Add user stories and other build-out of the expression attri… [DESIGN] Add user stories / build-out of the expression attributes May 14, 2024
@aphillips aphillips marked this pull request as ready for review May 14, 2024 23:37
@aphillips aphillips added syntax Issues related with syntax or ABNF design Design document or issues related to design labels May 15, 2024
@aphillips aphillips merged commit 8402d73 into main May 20, 2024
1 check passed
@aphillips aphillips deleted the aphillips-expression-attributes branch May 20, 2024 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design Design document or issues related to design syntax Issues related with syntax or ABNF
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants