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.

Sign up for GitHub

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

Jump to bottom

feat: interactive mode #854

Merged
merged 11 commits into from
Jan 14, 2024
Merged

feat: interactive mode #854

merged 11 commits into from
Jan 14, 2024

Conversation

AlexsJones
Copy link
Member

Closes #

📑 Description

✅ Checks

  • [x ] My pull request adheres to the code style of this project
  • [ x] My code requires changes to the documentation
  • [ x] I have updated the documentation as required
  • [x ] All the tests have passed

ℹ Additional Information

@AlexsJones
chore: wip interactive mode
7be7cb9 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
chore: tidied up a bit
c53f615 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones AlexsJones requested review from a team as code owners January 10, 2024 09:06
@AlexsJones AlexsJones changed the title feat/interactive mode feat: interactive mode Jan 10, 2024
@AlexsJones
chore: updated go mod
ef09ac1 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
chore: const prompt
4c40979 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
chore: resolved mod file
7a34b3b 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
matthisholleville
matthisholleville requested changes Jan 10, 2024
View reviewed changes
Copy link
Contributor

@matthisholleville matthisholleville left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only one comment. Good job.

cmd/analyze/analyze.go Show resolved Hide resolved
bwplotka
bwplotka approved these changes Jan 10, 2024
View reviewed changes
Copy link
Contributor

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏽 Good start, thanks!

cmd/analyze/analyze.go Outdated Show resolved Hide resolved
cmd/analyze/analyze.go Show resolved Hide resolved
cmd/analyze/analyze.go Outdated Show resolved Hide resolved
@AlexsJones
chore: updated based on comments feedback
0103013 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
chore: resolved conflicts
38b2aa7 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
matthisholleville
matthisholleville requested changes Jan 12, 2024
View reviewed changes
pkg/ai/interactive/interactive.go Outdated Show resolved Hide resolved
pkg/ai/interactive/interactive.go Outdated Show resolved Hide resolved
pkg/ai/interactive/interactive.go Outdated Show resolved Hide resolved
pkg/ai/interactive/interactive.go Outdated Show resolved Hide resolved
cmd/analyze/analyze.go Outdated Show resolved Hide resolved
cmd/analyze/analyze.go Outdated Show resolved Hide resolved
cmd/analyze/analyze.go Outdated Show resolved Hide resolved
@AlexsJones
chore: updated enum
6102279 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
chore: updated enum
d4a7a62 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
chore: merged main
4cc4785 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
matthisholleville
matthisholleville approved these changes Jan 12, 2024
View reviewed changes
Copy link
Contributor

@matthisholleville matthisholleville left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlexsJones AlexsJones enabled auto-merge (rebase) January 14, 2024 20:21
@AlexsJones AlexsJones merged commit 9da75e0 into main Jan 14, 2024
10 checks passed
@AlexsJones AlexsJones deleted the feat/interactive-mode branch January 14, 2024 20:22
@github-actions github-actions bot mentioned this pull request Jan 14, 2024
Merged
@Porkepix Porkepix mentioned this pull request Jan 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bwplotka bwplotka bwplotka approved these changes

@matthisholleville matthisholleville matthisholleville approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Backlog
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@AlexsJones @bwplotka @matthisholleville