Skip to content

fix changing global defaults - #1576

Merged
UziTech merged 2 commits into
markedjs:masterfrom
UziTech:fix-defaults
Dec 5, 2019
Merged

fix changing global defaults#1576
UziTech merged 2 commits into
markedjs:masterfrom
UziTech:fix-defaults

Conversation

@UziTech

Copy link
Copy Markdown
Member

Marked version: master

Description

fix regression in setOptions being global

Contributor

  • Test(s) exist to ensure functionality and minimize regression

Committer

In most cases, this should be a different person than the contributor.

  • Draft GitHub release notes have been updated.
  • CI is green (no forced merge required).
  • Merge PR

@vercel

vercelBot commented Dec 5, 2019

Copy link
Copy Markdown

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/markedjs/markedjs/d23zmogda
🌍 Preview: https://markedjs-git-fork-uzitech-fix-defaults.markedjs.now.sh

@styflestyfle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks!

@UziTech
UziTech merged commit ef7fa93 into markedjs:masterDec 5, 2019
@UziTech
UziTech deleted the fix-defaults branch December 5, 2019 23:03
@UziTechUziTech mentioned this pull request Dec 5, 2019
Merged
12 tasks
This was referenced Dec 20, 2019
Sign up for freeto 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

@UziTech@styfle