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
Adds two new CLI features - search and AI search #13611
Conversation
Changelog[uncommitted] (2023-08-21)Features
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know this is a non-functional draft PR, but I gave it a quick high-level review.
c33e668
to
261fb28
Compare
New version looks much better. No obvious style/design issues to point out. |
5f802a6
to
7630544
Compare
cbe4499
to
a2ead05
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Primarily looked at API usage and types. You can simplify your API types since you only need to worry about deserialization.
63dd709
to
1424dd2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For CLI command changes, it'd be awesome to see a screenshot (or gif or https://asciinema.org) in the PR description showing off the usage.
Some comments, but LGTM otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Search API bits LGTM!
Adds initial (not-totally-functional) pulumi search implementation
Still TODO: - Validate users' ability to call this endpoint - Iterate on naive search syntax to be a bit more compatible with Pulumi Finalize interface
Removes debugging print statements Remaining TODO: - Decouple this command from the context of a Pulumi project - Improve rendering logic to make results more readable - Add user interaction for things like pagination, etc
Moves them to only the httpstate backend to reduce cross-contamination
1424dd2
to
3960da2
Compare
Adds initial pulumi search implementation
Description
Fixes #13489
Fixes #12868
Checklist
make tidy
to update any new dependenciesmake lint
to verify my code passes the lint checkgofumpt
make changelog
and committed thechangelog/pending/<file>
documenting my change