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

Extend API for directives and attributes #49

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

Sidnioulz
Copy link
Owner

This branch extends API support for creating and search directives and attributes. As part of that work, ForNode/ForDirective was heavily normalised through compiler transforms, leading to an AST now distinct from that produced by Vue 3's SFC compiler.

Work to do:

  • createAttribute (already done before)
  • createDirective
  • Typed utilities for individual directives
  • ForNode normalisation
  • Attribute creation unit tests
  • Directive creation unit tests
  • ForDirective unit tests
  • ForNode unit tests (:construction:)
  • Other relevant API unit tests (:construction:)
  • Documentation for ForDirective and ForNode changes

@Sidnioulz Sidnioulz added the enhancement New feature or request label Oct 2, 2023
@Sidnioulz Sidnioulz added this to the v1 milestone Oct 2, 2023
@Sidnioulz Sidnioulz self-assigned this Oct 2, 2023
@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

Attention: 108 lines in your changes are missing coverage. Please review.

Comparison is base (9c23bc6) 78.45% compared to head (c7c3639) 81.07%.
Report is 12 commits behind head on main.

❗ Current head c7c3639 differs from pull request most recent head 9bef4ac. Consider uploading reports for the commit 9bef4ac to get more accurate results

Files Patch % Lines
src/template/api.ts 82.48% 61 Missing and 1 partial ⚠️
src/shims/compiler-core.d.ts 0.00% 24 Missing ⚠️
src/template/stringify.ts 87.61% 14 Missing ⚠️
src/template/utils.ts 90.62% 3 Missing ⚠️
src/transformTemplate.ts 70.00% 3 Missing ⚠️
src/template/compile.ts 97.77% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
+ Coverage   78.45%   81.07%   +2.62%     
==========================================
  Files          20       22       +2     
  Lines        1467     1982     +515     
  Branches      163      229      +66     
==========================================
+ Hits         1151     1607     +456     
- Misses        313      371      +58     
- Partials        3        4       +1     

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant