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.

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

Improvements to SLIM-CLI #6

Merged
merged 5 commits into from
Aug 28, 2024
Merged

Improvements to SLIM-CLI #6

merged 5 commits into from
Aug 28, 2024

Conversation

riverma
Copy link
Collaborator

@riverma riverma commented Aug 28, 2024

Purpose

  • Several improvements to the CLI. See below.

Proposed Changes

  • [ADD] Unit tests
  • [ADD] Forking / custom-remote support for deploy or apply-deploy (you can push to remotes other than origin now)
  • [ADD] Repos can be specified inside a file rather than just command line
  • [CHANGE] Prompt language for AI for better accuracy, specifically for Continuous Testing. Before, the prompt would create (within a TESTING.md) test descriptions that don't yet exist. Now, the prompt only creates test descriptions that exist currently in a repository. This is far more useful to projects.

Issues

  • N/A

Testing

  • Tested locally successfully. AI prompt changes mean more accuracy for continuous testing and other features work well.

@riverma riverma self-assigned this Aug 28, 2024
@riverma riverma added the enhancement New feature or request label Aug 28, 2024
Copy link
Contributor

@nutjob4life nutjob4life left a comment

Choose a reason for hiding this comment

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

LGTM

@yunks128
Copy link
Contributor

@riverma Thank you for the update!

[CHANGE] Prompt language for AI for better accuracy, specifically for Continuous Testing. Before, the prompt would create (within a TESTING.md) test descriptions that don't yet exist. Now, the prompt only creates test descriptions that exist currently in a repository. This is far more useful to projects.

Thanks for tackling this issue! It looks great!

@riverma
Copy link
Collaborator Author

riverma commented Aug 28, 2024

Thank you @nutjob4life @yunks128 for the quick review!

@riverma riverma merged commit 36a42ec into main Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants