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

Clean up marked config #2324

Merged
merged 2 commits into from
Aug 19, 2022
Merged

Clean up marked config #2324

merged 2 commits into from
Aug 19, 2022

Conversation

36degrees
Copy link
Contributor

See individual commits for details.

The `tables` option was removed in Marked v0.7.0 [1] so this doesn’t do anything any more.

[1]: https://github.com/markedjs/marked/releases/tag/v0.7.0
The documentation for the `pedantic` option [1] says:

> If true, conform to the original markdown.pl as much as possible. Don't fix original markdown bugs or behavior. Turns off and overrides gfm.

Verified that this option is doing nothing by comparing the build output with and without this setting, and there is no difference (other than the obfuscated email address on the community page, which is non-deterministic and so changes with every build):

```
$ diff -r deploy/public-marked-main deploy/public
diff -r deploy/public-marked-main/community/design-system-working-group/index.html deploy/public/community/design-system-working-group/index.html
1045c1045
< <p>In the meantime, if you are interested in joining the working group email the Design System team at <a href="mailto:&#103;&#111;&#x76;&#117;&#x6b;&#x2d;&#x64;&#101;&#115;&#105;&#x67;&#110;&#x2d;&#115;&#x79;&#115;&#116;&#101;&#x6d;&#x2d;&#115;&#x75;&#x70;&#x70;&#x6f;&#114;&#x74;&#x40;&#100;&#x69;&#x67;&#105;&#116;&#97;&#x6c;&#46;&#99;&#x61;&#x62;&#105;&#110;&#101;&#x74;&#45;&#x6f;&#x66;&#x66;&#105;&#99;&#x65;&#46;&#103;&#x6f;&#x76;&#x2e;&#117;&#x6b;">&#103;&#111;&#x76;&#117;&#x6b;&#x2d;&#x64;&#101;&#115;&#105;&#x67;&#110;&#x2d;&#115;&#x79;&#115;&#116;&#101;&#x6d;&#x2d;&#115;&#x75;&#x70;&#x70;&#x6f;&#114;&#x74;&#x40;&#100;&#x69;&#x67;&#105;&#116;&#97;&#x6c;&#46;&#99;&#x61;&#x62;&#105;&#110;&#101;&#x74;&#45;&#x6f;&#x66;&#x66;&#105;&#99;&#x65;&#46;&#103;&#x6f;&#x76;&#x2e;&#117;&#x6b;</a>.</p>
---
> <p>In the meantime, if you are interested in joining the working group email the Design System team at <a href="mailto:&#x67;&#111;&#118;&#117;&#x6b;&#x2d;&#100;&#101;&#x73;&#105;&#103;&#x6e;&#45;&#115;&#x79;&#x73;&#116;&#101;&#109;&#x2d;&#115;&#x75;&#112;&#112;&#x6f;&#114;&#116;&#x40;&#100;&#105;&#x67;&#105;&#116;&#97;&#x6c;&#x2e;&#x63;&#97;&#x62;&#x69;&#110;&#101;&#116;&#x2d;&#x6f;&#102;&#102;&#x69;&#99;&#x65;&#46;&#103;&#111;&#118;&#x2e;&#117;&#107;">&#x67;&#111;&#118;&#117;&#x6b;&#x2d;&#100;&#101;&#x73;&#105;&#103;&#x6e;&#45;&#115;&#x79;&#x73;&#116;&#101;&#109;&#x2d;&#115;&#x75;&#112;&#112;&#x6f;&#114;&#116;&#x40;&#100;&#105;&#x67;&#105;&#116;&#97;&#x6c;&#x2e;&#x63;&#97;&#x62;&#x69;&#110;&#101;&#116;&#x2d;&#x6f;&#102;&#102;&#x69;&#99;&#x65;&#46;&#103;&#111;&#118;&#x2e;&#117;&#107;</a>.</p>
```

[1]: https://marked.js.org/using_advanced#options
@netlify
Copy link

netlify bot commented Aug 19, 2022

You can preview this change here:

Name Link
🔨 Latest commit 4811829
🔍 Latest deploy log https://app.netlify.com/sites/govuk-design-system-preview/deploys/62ff91aac31bd60008c971be
😎 Deploy Preview https://deploy-preview-2324--govuk-design-system-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@owenatgov owenatgov left a comment

Choose a reason for hiding this comment

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

Makes sense to me 👍🏻

@36degrees 36degrees merged commit f2f34ea into main Aug 19, 2022
@36degrees 36degrees deleted the marked-config branch August 19, 2022 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants