Skip to content

WIP: Showdown config overrides#17

Open
vi-cat wants to merge 8 commits intoSnowdogApps:masterfrom
vi-cat:feature/mdParserOptions
Open

WIP: Showdown config overrides#17
vi-cat wants to merge 8 commits intoSnowdogApps:masterfrom
vi-cat:feature/mdParserOptions

Conversation

@vi-cat
Copy link

@vi-cat vi-cat commented Mar 24, 2020

  • Added the ability to let users override showdown's config through wombat's config.
  • Refactored get-config.js to be a singleton

@vi-cat vi-cat changed the title initial effort WIP: Showdown config overrides Mar 24, 2020
@vi-cat vi-cat force-pushed the feature/mdParserOptions branch from 8be0ea9 to 660b732 Compare March 25, 2020 09:00
@vi-cat vi-cat force-pushed the feature/mdParserOptions branch from 660b732 to 5f0154d Compare March 25, 2020 09:00
@vi-cat
Copy link
Author

vi-cat commented Mar 25, 2020

Unit tests seem to be broken (as they are in master too), I will take a look.

@Igloczek
Copy link
Contributor

Unit tests seem to be broken (as they are in master too), I will take a look.

Yeah, that's unfortunately true, I didn't update them for some time, sorry about that :(

@vi-cat vi-cat force-pushed the feature/mdParserOptions branch 3 times, most recently from 92a7f9d to cdbc36b Compare March 25, 2020 10:20
@vi-cat vi-cat force-pushed the feature/mdParserOptions branch from cdbc36b to 7ed8d70 Compare March 25, 2020 10:20
@vi-cat vi-cat force-pushed the feature/mdParserOptions branch from da28207 to 5f5d80c Compare March 25, 2020 15:16
@vi-cat
Copy link
Author

vi-cat commented Mar 25, 2020

@Igloczek I fixed the tests and the config overriding appears to work for me. Do you want to test my changes locally before merging?

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.

2 participants