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

Add test utils, logger improvements, general polish #12

Merged
merged 35 commits into from
Jun 26, 2024
Merged

Conversation

ryangoree
Copy link
Owner

  • Polished up internal ✨test utils✨ and added them to the exports so you can easily test you're CLIs and plugins!
  • Made the logger plugin more full featured with prefix, logFile, and enabled options, and util functions for enabling/disabling the logger during runtime.
  • Added formatFileName util function to exports.
  • Added getters to State for client, ... so they can be accessed without needing to go through through Context.
  • Made the Hooks type a generic which takes a HooksObject type param.
  • Made the PluginInfo type a generic which takes a PluginMeta type param.
  • Renamed StateOptions.data to StateOptions.initialData.
  • Refactored command resolution to ignore relative paths.
  • Fixed the optionValues type on State.fork to work with command types that have an optional options field.
  • Renamed skip option to enabled in command menu plugin

Copy link

changeset-bot bot commented Jun 26, 2024

🦋 Changeset detected

Latest commit: fa70899

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

This PR includes changesets to release 2 packages
Name Type
clide-plugin-command-menu Major
clide-js Minor

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

@ryangoree ryangoree merged commit 2eb1489 into main Jun 26, 2024
3 checks passed
@ryangoree ryangoree deleted the ryan-patches branch June 26, 2024 15:38
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