Skip to content

Conversation

@jaredwray
Copy link
Owner

@jaredwray jaredwray commented Mar 29, 2025

Please check if the PR fulfills these requirements

  • Followed the Contributing guidelines.
  • Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
fix: updating jsDelivr url to use latest

@jaredwray jaredwray requested a review from Copilot March 29, 2025 20:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates the jsDelivr URL in the README to use the latest version of the Writr package.

  • Removed the version number from the jsDelivr import URL to always pull the most recent version.
  • Ensured that the code sample in the README reflects this change.
Comments suppressed due to low confidence (1)

README.md:112

  • Removing the version pin means that future updates to Writr might break this example unexpectedly. Consider whether this behavior is desired or if a version should be explicitly specified for stability.
import {Writr} from 'https://cdn.jsdelivr.net/npm/writr/+esm'

@codecov
Copy link

codecov bot commented Mar 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (59405e8) to head (579cb9d).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #345   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines          373       373           
  Branches        88        88           
=========================================
  Hits           373       373           

☔ 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.

@jaredwray jaredwray merged commit 2287a89 into main Mar 29, 2025
7 checks passed
@jaredwray jaredwray deleted the fix-updating-jsDelivr-url-to-use-latest branch March 29, 2025 20:49
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