Skip to content

Add resource contention considerations - #765

Merged
anssiko merged 2 commits into
mainfrom
tag-resource-contention
Oct 24, 2024
Merged

Add resource contention considerations#765
anssiko merged 2 commits into
mainfrom
tag-resource-contention

Conversation

@anssiko

@anssiko anssiko commented Oct 10, 2024

Copy link
Copy Markdown
Member

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

anssiko commented Oct 17, 2024

Copy link
Copy Markdown
Member Author

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

@fdwr fdwr left a comment

Copy link
Copy Markdown
Collaborator

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.

Comment thread index.bs Outdated
Co-authored-by: Dwayne Robinson <dwayner@microsoft.com>
@anssiko

anssiko commented Oct 24, 2024

Copy link
Copy Markdown
Member Author

@fdwr thank you for the review and grammar suggestions.

@anssiko

anssiko commented Oct 24, 2024

Copy link
Copy Markdown
Member Author

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
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

@huningxin huningxin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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