Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add resource contention considerations #765

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Conversation

anssiko
Copy link
Member

@anssiko anssiko commented Oct 10, 2024

This is a first stab to port comments from @RafaelCintron @reillyeon in the TAG issue w3ctag/design-reviews#933 to the specification to address feedback from the TAG. Please suggest changes to make sure key points are captured.


Preview | Diff

This is to address feedback from the TAG review:
w3ctag/design-reviews#933
@anssiko
Copy link
Member Author

anssiko commented Oct 17, 2024

PTAL @RafaelCintron, I appreciate a review from you because this in part builds upon a discussion you had with the TAG.

Copy link
Collaborator

@fdwr fdwr left a comment

Choose a reason for hiding this comment

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

Approved with grammar suggestions. @RafaelCintron may have thoughts too.

Co-authored-by: Dwayne Robinson <dwayner@microsoft.com>
@anssiko
Copy link
Member Author

anssiko commented Oct 24, 2024

@fdwr thank you for the review and grammar suggestions.

@anssiko
Copy link
Member Author

anssiko commented Oct 24, 2024

I'll merge this PR now to focus our open PR queue. I'll also inform the TAG that we've addressed their feedback.

@anssiko anssiko merged commit 2cc59c5 into main Oct 24, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Oct 24, 2024
SHA: 2cc59c5
Reason: push, by anssiko

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@anssiko anssiko deleted the tag-resource-contention branch October 24, 2024 14:00
Copy link
Contributor

@huningxin huningxin left a comment

Choose a reason for hiding this comment

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

LGTM!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants