player-ui / tools

A Place for tooling related to Player
MIT License
9 stars 12 forks source link

Remove Unneeded Dependency #178

Closed KetanReddy closed 1 week ago

KetanReddy commented 1 week ago

Release Notes

Change Type (required)

Indicate the type of change your pull request is:

📦 Published PR as canary version: 0.8.2--canary.178.3944
Try this version out locally by upgrading relevant packages to 0.8.2--canary.178.3944

Version

Published prerelease version: 0.9.0-next.3

Changelog ### Release Notes #### Remove Unneeded Dependency ([#178](https://github.com/player-ui/tools/pull/178)) - Remove unneeded dependency `@oclif/plugin-legacy` - Fix issue with how the DSL compilation command passed arguments to the JSON validation plugin ### Change Type (required) Indicate the type of change your pull request is: - [x] `patch` - [ ] `minor` - [ ] `major` #### Add options to LSPAssetsPlugin to load from TSManifest via module import ([#171](https://github.com/player-ui/tools/pull/171)) Allow for loading XLRs to LSP via module imports. Add explicit (optional for now) toggle for distinguishing between module and manifest loading. --- #### 🚀 Enhancement - Multiple devtools plugin fix [#174](https://github.com/player-ui/tools/pull/174) ([@mercillo](https://github.com/mercillo)) #### 🐛 Bug Fix - Remove Unneeded Dependency [#178](https://github.com/player-ui/tools/pull/178) ([@KetanReddy](https://github.com/KetanReddy)) - feat: complexity checker plugin [#169](https://github.com/player-ui/tools/pull/169) ([@KetanReddy](https://github.com/KetanReddy) [@kharrop](https://github.com/kharrop) [@cehan-Chloe](https://github.com/cehan-Chloe) [@rafbcampos](https://github.com/rafbcampos) [@intuit-svc](https://github.com/intuit-svc) [@mercillo](https://github.com/mercillo)) - Add options to LSPAssetsPlugin to load from TSManifest via module import [#171](https://github.com/player-ui/tools/pull/171) ([@KetanReddy](https://github.com/KetanReddy)) #### Authors: 6 - [@intuit-svc](https://github.com/intuit-svc) - Chloeeeeeee ([@cehan-Chloe](https://github.com/cehan-Chloe)) - Kelly Harrop ([@kharrop](https://github.com/kharrop)) - Ketan Reddy ([@KetanReddy](https://github.com/KetanReddy)) - Marlon "Marky" Ercillo ([@mercillo](https://github.com/mercillo)) - Rafael Campos ([@rafbcampos](https://github.com/rafbcampos))
codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.25%. Comparing base (9af842c) to head (1f3ea61). Report is 3 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #178 +/- ## ======================================= Coverage 73.25% 73.25% ======================================= Files 85 85 Lines 12876 12876 Branches 1201 1201 ======================================= Hits 9432 9432 Misses 3423 3423 Partials 21 21 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features:

KetanReddy commented 1 week ago

/canary