Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

LLM-1387: remove deprecation warning#58

Merged
Zilv1nas merged 1 commit into
mainfrom
remove-deprecation-warning
May 6, 2026
Merged

LLM-1387: remove deprecation warning#58
Zilv1nas merged 1 commit into
mainfrom
remove-deprecation-warning

Conversation

@Zilv1nas
Copy link
Copy Markdown
Contributor

@Zilv1nas Zilv1nas commented May 6, 2026


Kimchi Summary

What changed

Removes the CLI deprecation warnings from scripts/install.sh. The script continues to redirect users to the kimchi-dev installer but no longer displays the yellow warning banner or blue redirect notice.

Key changes

  • scripts/install.sh: Removed color code variables (YELLOW, BLUE, NC) and echo statements that printed the deprecation warning. The exec bash redirect to the kimchi-dev release remains unchanged.

Impact

Users executing the install script will no longer see deprecation warnings before being redirected to the new installer.

@kimchi-review
Copy link
Copy Markdown

kimchi-review Bot commented May 6, 2026

Kimchi Code Review

Property Value
Commit 2fb6008
Author @Zilv1nas
Files changed 0
Review status Completed
Comments 0
Duration 45s

Summary

📊 Review Score: 95/100 (overall code quality — 0 lowest, 100 highest)
⏱️ Estimated effort to review: 1/5 (1 = trivial, 5 = very complex)

🧪 Tests: no — No tests were added or modified for this change. The removal of deprecation warning messaging and color variables in the bash script does not include accompanying test updates.

No significant issues found. LGTM! 🎉

What to expect

Kimchi will analyze the changes in this pull request and post:

  • A summary of the overall changes
  • Inline comments on specific lines with findings categorized by issue type

The review typically completes within a few minutes. This comment will be updated once the review is ready.

Interact with Kimchi
  • @kimchi review — re-trigger a full review on the latest commit
  • @kimchi summary — regenerate the PR summary
  • @kimchi ignore — skip this PR (no review will be posted)
  • Reply to any inline comment to ask follow-up questions or request clarification
Configuration

Reviews are configured by your organization admin.
Review instructions, excluded directories, and severity thresholds can be adjusted per repository in the Kimchi dashboard.


Powered by Kimchi — AI-powered code review by CAST AI

Copy link
Copy Markdown

@kimchi-review kimchi-review Bot left a comment

Choose a reason for hiding this comment

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

📊 Review Score: 95/100 (overall code quality — 0 lowest, 100 highest)
⏱️ Estimated effort to review: 1/5 (1 = trivial, 5 = very complex)

🧪 Tests: no — No tests were added or modified for this change. The removal of deprecation warning messaging and color variables in the bash script does not include accompanying test updates.

No significant issues found. LGTM! 🎉

@Zilv1nas Zilv1nas merged commit c59487e into main May 6, 2026
2 checks passed
@Zilv1nas Zilv1nas deleted the remove-deprecation-warning branch May 6, 2026 15:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants