Skip to content

Conversation

trolzen
Copy link
Contributor

@trolzen trolzen commented Oct 17, 2025

I've also updated the original page:

  1. The --attribute option requires its value to be an equal sign-separated pair.
  2. In the last example, pdf and asciidoctor-pdf shouldn't be marked as placeholders.
  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The page(s) follow the style guide.
  • The PR contains at most 5 new pages.
  • The PR title conforms to the recommended templates.

@github-actions github-actions bot added page edit Changes to an existing page(s). review needed Prioritized PRs marked for reviews from maintainers. translation edit Changes to an existing translation page(s). labels Oct 17, 2025
- Convert a specific `.adoc` file to HTML and link a CSS stylesheet:

`asciidoctor {{[-a|--attribute]}} stylesheet {{path/to/stylesheet.css}} {{path/to/file.adoc}}`
`asciidoctor {{[-a|--attribute]}} stylesheet={{path/to/stylesheet.css}} {{path/to/file.adoc}}`
Copy link
Member

Choose a reason for hiding this comment

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

Wait this isn't an option. The = can stay. Does this command even work without =?

Copy link
Contributor Author

@trolzen trolzen Oct 18, 2025

Choose a reason for hiding this comment

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

Here --attribute is the option, and stylesheet={{path/to/stylesheet.css}} is its value, the whole thing. And the documentation explicitly states that it be in this form with an equal sign for key-value attributes (see the link). So I don't think it would accept space there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've tested it a bit and can confirm, that equal sign is mandatory here. Without it asciidoctor treats the stylesheet file as a source file that need to be converted, not a stylesheet file.

Copy link
Member

Choose a reason for hiding this comment

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

You should change the page in that case

Copy link
Contributor

@iridacea iridacea left a comment

Choose a reason for hiding this comment

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

Looks good to me.

- Convert a specific `.adoc` file to HTML and link a CSS stylesheet:

`asciidoctor {{[-a|--attribute]}} stylesheet {{path/to/stylesheet.css}} {{path/to/file.adoc}}`
`asciidoctor {{[-a|--attribute]}} stylesheet={{path/to/stylesheet.css}} {{path/to/file.adoc}}`
Copy link
Member

Choose a reason for hiding this comment

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

You should change the page in that case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

page edit Changes to an existing page(s). review needed Prioritized PRs marked for reviews from maintainers. translation edit Changes to an existing translation page(s).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants