Skip to content

fix: remove dependency on @babel/types#181

Merged
rturnq merged 1 commit intomainfrom
remove-babel-types
Feb 6, 2026
Merged

fix: remove dependency on @babel/types#181
rturnq merged 1 commit intomainfrom
remove-babel-types

Conversation

@rturnq
Copy link
Contributor

@rturnq rturnq commented Feb 5, 2026

Use babel types from @marko/compiler instead of directly from @babel/types

@changeset-bot
Copy link

changeset-bot bot commented Feb 5, 2026

🦋 Changeset detected

Latest commit: 4c6c90b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@marko/run Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Walkthrough

The changes involve removing a dependency on Babel types and replacing it with types sourced from the Marko compiler. A new changeset file documents a patch version update. The package.json file removes the Babel types devDependency. The ast.ts file's import statement is updated to source types from the Marko compiler instead of Babel types, while the rest of the code remains functionally unchanged.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: removing the @babel/types dependency by switching to types from @marko/compiler.
Description check ✅ Passed The description is directly related to the changeset, explaining the rationale for using babel types from @marko/compiler instead of @babel/types.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-babel-types

Comment @coderabbitai help to get the list of available commands and usage tips.

@rturnq rturnq merged commit e20eba4 into main Feb 6, 2026
16 of 17 checks passed
@rturnq rturnq deleted the remove-babel-types branch February 6, 2026 00:03
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (e3545dd) to head (4c6c90b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff     @@
##   main   #181   +/-   ##
===========================
===========================

☔ 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant