-
Notifications
You must be signed in to change notification settings - Fork 145
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
feat: Just-in-time plugin installation #8482
base: main
Are you sure you want to change the base?
feat: Just-in-time plugin installation #8482
Conversation
✅ Deploy Preview for meltano canceled.
|
93cc0b1
to
7b9bf26
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8482 +/- ##
==========================================
+ Coverage 92.01% 92.06% +0.05%
==========================================
Files 246 246
Lines 19380 19441 +61
Branches 3419 3436 +17
==========================================
+ Hits 17833 17899 +66
+ Misses 1271 1267 -4
+ Partials 276 275 -1 ☔ View full report in Codecov by Sentry. |
We probably want to suppress install progress to stdout from
|
Yeah, I'd be more than OK with starting to unify our output away from |
601322c
to
44680fa
Compare
Think I have covered all relevant commands here. Wondering if we want a new
I haven't yet addressed install progress/status as logs, but the output is suppressed for |
ad6e137
to
89f1848
Compare
Should this be added for |
If it's opt-in, probably yes 👍 |
3b7d4b7
to
21cb02d
Compare
11ca242
to
69f8295
Compare
Co-authored-by: Edgar Ramírez Mondragón <16805946+edgarrmondragon@users.noreply.github.com>
Implements just-in-time install for the following commands:
invoke
run
el
/elt
test
config <plugin> test
select --list