-
Notifications
You must be signed in to change notification settings - Fork 75
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
Add a prototype for asciidoc #41
base: master
Are you sure you want to change the base?
Changes from all commits
2a573f9
13930a7
3322f1a
7bb0290
3b3de51
ed62689
da40ee7
4e5beb2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
:revremark: State: {state} | ||
:attribute-missing: warn | ||
:docinfo: | ||
:doctype: article | ||
:hide-uri-scheme: | ||
:icons: font | ||
:idprefix: | ||
:idseparator: - | ||
:numbered: | ||
:sectanchors: | ||
:sectlinks: | ||
:sectnums: | ||
:sectnumlevels: 1 | ||
:showtitle: | ||
:source-highlighter: pygments | ||
:toc: left | ||
:xrefstyle: full | ||
ifdef::env-github[] | ||
:tip-caption: :bulb: | ||
:note-caption: :information_source: | ||
:important-caption: :heavy_exclamation_mark: | ||
:caution-caption: :fire: | ||
:warning-caption: :warning: | ||
endif::[] |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
--- | ||
authors: Han Solo <[email protected]>, Frodo Baggins <[email protected]> | ||
state: predraft | ||
--- | ||
= RFD <NUMBER> <Title> | ||
:author: Han Solo <[email protected]>; Frodo Baggins <[email protected]> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we need this duplication? If there is a reason for 'author' and 'state' in adoc attributes, then I say we should drop the jekyll-format frontmatter. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @trentm asciidoc does make use of this attribute somehow, somewhere. If you look at the rendered HTML, it has a placeholder for this value: The front-matter for jekyll is purely for Jekyll. asciidoc ignores it completely (and though does save the value for reference, the content of the front-matter is un-indexable by asciidoc thereby rendering it effectively useless to asciidoc). I'm indifferent to its presence despite the fact that it doesn't add any value to the meaning of the content and also doesn't improve Github's rendering of the document (compare GH's rendering of asciidoc vs the actual There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, the front matter is being parsed by the chat bot, and I think by 'make check'. I think I'd prefer we just use the asciidoc attributes, and drop the jekyll front-matter for .adoc RFDs. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I updated As for the chat bots, I haven't touched them but suspect it would be a similar change. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yup, thanks. I updated the new chatbot (joybot1999) already. I'll get the current jabber bot. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I lied, I won't get the current jabber bot. Meh, if we are going to move away from it in the next couple of weeks. |
||
:state: predraft | ||
:revnumber: 0.1.0 | ||
:keywords: rfd <NUMBER>, prototype, sample, keywords | ||
include::front-matter.adoc[] | ||
|
||
//// | ||
This Source Code Form is subject to the terms of the Mozilla Public | ||
License, v. 2.0. If a copy of the MPL was not distributed with this | ||
file, You can obtain one at http://mozilla.org/MPL/2.0/. | ||
|
||
Copyright 2017 <contribution> | ||
//// | ||
|
||
[[introduction]] | ||
== Introduction |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
blank link before the title please... or perhaps let's move the title to after all the adoc attributes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trentm np. I'm curious as to why there is a preference. I haven't read anything to suggest it matters to asciidoc.