Skip to content

Auto-publish the explainer explainer to TR space.#23

Merged
jyasskin merged 2 commits into
w3ctag:mainfrom
jyasskin:autopublish
Jun 18, 2025
Merged

Auto-publish the explainer explainer to TR space.#23
jyasskin merged 2 commits into
w3ctag:mainfrom
jyasskin:autopublish

Conversation

@jyasskin

@jyasskin jyasskin commented May 20, 2025

Copy link
Copy Markdown
Member

@ylafon, could you set up the Echidna secret before approving?

Fixes #13.


Preview | Diff

@jyasskin
jyasskin requested review from torgo and ylafon May 20, 2025 19:14

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In W3C_BUILD_OVERRIDE, the metadata order should be changed. While it makes sense to put the "Participate" links first in the edcopy, it is not the case for the published version.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also, "Participate:" and "Feedback:" are almost duplicates in the /TR/ version, so perhaps removing it and adding Participate (or Feedback) in the editor's copy workflow only would be good.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@ylafon ylafon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See comment about metadata ordering

@ylafon

ylafon commented Jun 12, 2025

Copy link
Copy Markdown
Member

@ylafon, could you set up the Echidna secret before approving?

Fixes #13.

Preview | Diff

ECHIDNA_TOKEN has been created

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@jyasskin
jyasskin merged commit 7aeea94 into w3ctag:main Jun 18, 2025
2 checks passed
@jyasskin
jyasskin deleted the autopublish branch June 18, 2025 17:18
github-actions Bot added a commit that referenced this pull request Jun 18, 2025
SHA: 7aeea94
Reason: push, by jyasskin

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.

Consider turning the Explainer Explainer into a Note

4 participants