Skip to content
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

Expand the API proposal to include a "behavior" section #582

Merged

Conversation

rdcronin
Copy link
Collaborator

@rdcronin rdcronin commented Apr 1, 2024

The behavior of certain APIs may not be immediately clear from the
API schema. Add a separate section for "behavior" to allow authors
to specify notes related to the behavior of the API if they are non-
obvious.

The behavior of certain APIs may not be immediately clear from the
API schema. Add a separate section for "behavior" to allow authors
to specify notes related to the behavior of the API if they are non-
obvious.
@rdcronin rdcronin requested a review from Rob--W April 1, 2024 21:54
@rdcronin
Copy link
Collaborator Author

rdcronin commented Apr 1, 2024

@Rob--W Please review -- let me know if this looks good to you, and I'll tag in a couple other folks.

I don't want to be too prescriptive in this section, but I do think it's worth having as part of the default proposal.

@rdcronin rdcronin requested review from dotproto and oliverdunk April 2, 2024 16:41
@rdcronin
Copy link
Collaborator Author

rdcronin commented Apr 2, 2024

Thanks, Rob and Timothy!

@dotproto and @oliverdunk , please also review.

If it looks good to you, please merge.

Copy link
Member

@dotproto dotproto left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@oliverdunk oliverdunk left a comment

Choose a reason for hiding this comment

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

LGTM - thanks!

@oliverdunk oliverdunk merged commit b0a84d4 into w3c:main Apr 3, 2024
3 checks passed
github-actions bot added a commit that referenced this pull request Apr 3, 2024
…or_section

SHA: b0a84d4
Reason: push, by oliverdunk

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Apr 3, 2024
…or_section

SHA: b0a84d4
Reason: push, by oliverdunk

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

5 participants