Skip to content

Add safeProtocol as a parameter for defaultUrlTransform#942

Open
janpe wants to merge 2 commits intoremarkjs:mainfrom
janpe:default-url-transform-protocol-parameter
Open

Add safeProtocol as a parameter for defaultUrlTransform#942
janpe wants to merge 2 commits intoremarkjs:mainfrom
janpe:default-url-transform-protocol-parameter

Conversation

@janpe
Copy link
Copy Markdown

@janpe janpe commented Apr 28, 2026

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and discussions and couldn’t find anything or linked relevant results below
  • I made sure the docs are up to date
  • I included tests (or that’s not needed)

Description of changes

Allow usage of defaultUrlTransform with custom safeProtocol regex string

Resolves #941

@github-actions github-actions Bot added 👋 phase/new Post is being triaged automatically 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Apr 28, 2026
@janpe janpe force-pushed the default-url-transform-protocol-parameter branch from 18f25d6 to 6c5e56d Compare April 28, 2026 04:57
@janpe janpe force-pushed the default-url-transform-protocol-parameter branch from 6c5e56d to 7af5938 Compare April 28, 2026 05:01
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (fda7fa5) to head (7af5938).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #942   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines         1743      1815   +72     
  Branches       123       131    +8     
=========================================
+ Hits          1743      1815   +72     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤞 phase/open Post is being triaged manually

Development

Successfully merging this pull request may close these issues.

Allow safeProtocol to be passed as a parameter for defaultUrlTransform

1 participant