-
-
Notifications
You must be signed in to change notification settings - Fork 12.4k
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
steampipe 0.20.2 #131342
steampipe 0.20.2 #131342
Conversation
|
Hi @chenrui333 We have published |
Thanks @binaek! :) |
767f33f
to
9469ab0
Compare
@binaek looks like still the same issue with the path setup. |
NOTE, this did happen before in #112068 |
Hi @chenrui333 , Looks like the version bump is still facing issues. We will be glad to help from our end! |
One of the efforts of the We found this in the Could this be the issue why the test is failing? |
@chenrui333 I have raised a PR with the source branch of this PR at chenrui333#9 The changes I am suggesting removes the need for the Formula to be tied down to an Error output - and asserting on the output of This has two advantages:
Can you please take a look? |
@binaek the issue is not really with test, the issue is more on the completion generation side |
I see what you are doing, let me update my test, we should not remove test though (good to have them to do some sanity checking) |
4712613
to
1afb5d0
Compare
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a |
@chenrui333 this PR has been marked as |
We are running It seems that running |
^^ @binaek |
@chenrui333 @alebcay that looks like a bug. Thank you for bringing it to our notice. I have opened an issue at turbot/steampipe#3485 to fix this. |
1afb5d0
to
6135277
Compare
ffe2d3c
to
db6fd3e
Compare
I am gonna disable the completion step and proceed with the upgrade. We can bring it back for the next release. |
Signed-off-by: Rui Chen <rui@chenrui.dev> steampipe: update test Signed-off-by: Rui Chen <rui@chenrui.dev>
db6fd3e
to
336bd0d
Compare
looks like the path issue got fixed, let's try it out in the new PR. |
Should be good now with #132379 |
Created by
brew bump
Created with
brew bump-formula-pr
.release notes
Mod Dependency Management
https
prefix. (#3257)requires
property. (#3347)Caching
meta_cache()
andmeta_cache_ttl()
. (#3442)What's new?
--progress
flag toplugin install
to disable progress bars. (#2953)Bug fixes
invalid output format
error when running dashboard ifoutput
is set in terminal options. (#3293)closes chenrui333#9