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

share: isso.conf: Use dashed-case for misaka 2.0 #685

Merged
merged 1 commit into from Feb 7, 2021

Conversation

ix5
Copy link
Collaborator

@ix5 ix5 commented Jan 17, 2021

Misaka expects extensions and flags in dashed-case instead of the legacy snake_case as of version 2.0.

See https://github.com/FSX/misaka/blob/v2.1.1/misaka/utils.py#L15-L37

Also align "options" values with the defaults in utils/html.py.

Misaka expects extensions and flags in `dashed-case` instead
of the legacy `snake_case` as of version 2.0.

See https://github.com/FSX/misaka/blob/v2.1.1/misaka/utils.py#L15-L37

Also align "options" values with the defaults in
`utils/html.py`.
options = strikethrough, autolink, fenced_code, no_intra_emphasis
# Misaka-specific Markdown extensions, all extensions can be used here,
# separated by comma, either by their name or by EXT_<extension>.
# Careful: Misaka 1.0 used "snake_case", but 2.0 needs "dashed-case"!
Copy link
Collaborator

Choose a reason for hiding this comment

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

We've been on misaka 2.0 since 1.5, do we really need to mention misaka 1.0?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I was directing that comment towards people upgrading from older versions.

@jelmer jelmer merged commit 72e624b into posativ:master Feb 7, 2021
1 check failed
@ix5 ix5 deleted the conf-misaka branch February 10, 2021 13:45
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.

None yet

2 participants