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

Use *-snapshot.svg logos for Commit Snapshots #309

Closed
wants to merge 1 commit into from
Closed

Conversation

annevk
Copy link
Member

@annevk annevk commented Mar 18, 2020

Closes whatwg/meta#88.

@annevk
Copy link
Member Author

annevk commented Mar 18, 2020

A variant of this worked locally, but haven't tested this yet. Reasonable approach or should we pass this information into Bikeshed somehow?

@domenic
Copy link
Member

domenic commented Mar 18, 2020

It's not terrible, but I think it'd be better to do this similar to how we do RDs, which is https://github.com/tabatkins/bikeshed/blob/master/bikeshed/boilerplate/whatwg/computed-metadata-RD.include . It might be as simple as adding computed-metadata-LS-COMMIT.include.

(Side note: I think we could clean up a good amount of the snapshot infrastructure using similar computed metadata techniques that make it similar to RDs.

curlbikeshed "$COMMIT_DIR/index.html" \
-F md-status=LS-COMMIT \
-F md-warning="Commit $SHA $COMMIT_URL_BASE$SHA replaced by $LS_URL" \
-F md-title="$H1 Standard (Commit Snapshot $SHA)" \
-F md-Text-Macro="SNAPSHOT-LINK $BACK_TO_LS_LINK"
could probably be replaced with -F md-status=LS-COMMIT -F md-Text-Macro="SHA $SHA".)

annevk added a commit that referenced this pull request Mar 27, 2020
They will now use a title aligned with Review Drafts, use the snapshot logo, and generally include less data that is best looked for in the Living Standard.

Depends upon speced/bikeshed#1635.

Closes whatwg/meta#88. Closes #309.
@annevk annevk closed this in #310 Apr 6, 2020
annevk added a commit that referenced this pull request Apr 6, 2020
Buids upon speced/bikeshed#1635. Remaining work tracked by whatwg/spec-factory#5.

Closes whatwg/meta#88. Closes #309.

Co-authored-by: Domenic Denicola <d@domenic.me>
@annevk annevk deleted the annevk-patch-1 branch April 22, 2020 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Use snapshot logos for commit-snapshots favicons
3 participants