Skip to content

Conversation

@gandazgul
Copy link

@gandazgul gandazgul commented Jul 14, 2024

Added an option uppercaseFuncNames that defaults to false, if set to true, it will compile functions that start with an uppercase letter and have props with object destructuring.

pnpm test

> [email protected] test /babel-plugin-solid-undestructure
> uvu tests

add-merge-props-call.test.js
•   (1 / 1)

check-func-capital-letter.test.js
•   (1 / 1)

check-func.test.js
•   (1 / 1)

get-props-info.test.js
•   (1 / 1)

handle-remaining-plugin-imports.test.js
•   (1 / 1)

index.test.js
•   (1 / 1)

remove-wrapping-ctf.test.js
•   (1 / 1)

transform-prop-references.test.js
•   (1 / 1)

validate-func.test.js
•   (1 / 1)

  Total:     9
  Passed:    9
  Skipped:   0
  Duration:  184.82ms

@what-the-diff
Copy link

what-the-diff bot commented Jul 14, 2024

PR Summary

  • Update .gitignore File
    The file that controls what gets tracked by our version control system has been updated to ignore certain types of files related to your IDE settings.

  • Expand playground.cjs with New Component Examples
    We have added examples of SolidJS and TypeScript components which serves as useful references when developing similar components in the future.

  • Improvements in the check-func.cjs File
    There are enhancements in our code checking functionality: a new function, namely checkFuncStartsWithUppercase, has been introduced. Also, checkFunc is now capable of handling functions that begin with uppercase characters, increasing the range of code it can help evaluate.

  • Refinement in the utils.cjs File
    Our utility function compares values in a more robust and reliable way with '===' now.

  • Addition of check-func-capital-letter.test.js File
    This new file includes test cases specifically for checkFunc, contributing to our confidence that it will function as expected by validating it in multiple scenarios.

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