Skip to content

[ Enhancement ] Adding Markdown preprocessor #15323

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

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

juanchax
Copy link

Description

Updated external url references contained in /public/content/contributing/index.md to {variable}
Populated variables.ts with external url references contained in /public/content/contributing/index.md
Added preprocess.ts with the markdown preprocessing logic
Removed preprocessMarkdown() from compile.ts
Imported preprocessMarkdown() from preprocess.ts

Related Issue

Background for this enhabcement --> #15312

Base issues this enhancement is attempting to resolve and prevent future ones -->

#14605

#14200

#14711

juanchax and others added 3 commits April 20, 2025 15:22
Populated variables.ts with external url references contained in /public/content/contributing/index.md
Added preprocess.ts with the markdown preprocessing logic
Removed preprocessMarkdown() from compile.ts
imported preprocessMarkdown() from preprocess.ts
@github-actions github-actions bot added content 🖋️ This involves copy additions or edits tooling 🔧 Changes related to tooling of the project labels Apr 20, 2025
Copy link

netlify bot commented Apr 20, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 1322fe0
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/680674ffbbb7d500086ffac7
😎 Deploy Preview https://deploy-preview-15323--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 46 (🔴 down 3 from production)
Accessibility: 95 (🟢 up 1 from production)
Best Practices: 89 (🔴 down 9 from production)
SEO: 98 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

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

added type guard for current escapeAnchorChars() implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits tooling 🔧 Changes related to tooling of the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant