Skip to content

Conversation

@efeecllk
Copy link

  • Fix "seperate" -> "separate" in ayrshare.py comment
  • Fix "noticibly" -> "noticeably" in tests.md documentation
  • Fix "funciton" -> "function" in tests.md documentation

0ubbe and others added 2 commits December 18, 2025 19:04
…vitas#11639)

### Changes 🏗️

Chat should be disabled by default; otherwise, it flashes, and if Launch
Darkly fails to fail, it is dangerous.

### Checklist 📋

#### For code changes:
- [x] I have clearly listed my changes in the PR description
- [x] I have made a test plan
- [x] I have tested my changes according to the test plan:
  - [x] Run locally with Launch Darkly disabled and test the above
- Fix "seperate" -> "separate" in ayrshare.py comment
- Fix "noticibly" -> "noticeably" in tests.md documentation
- Fix "funciton" -> "function" in tests.md documentation

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@efeecllk efeecllk requested a review from a team as a code owner December 30, 2025 15:04
@efeecllk efeecllk requested review from Bentlybro and kcze and removed request for a team December 30, 2025 15:04
@github-project-automation github-project-automation bot moved this to 🆕 Needs initial review in AutoGPT development kanban Dec 30, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 30, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@CLAassistant
Copy link

CLAassistant commented Dec 30, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ 0ubbe
❌ efeecllk
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot added documentation Improvements or additions to documentation platform/backend AutoGPT Platform - Back end labels Dec 30, 2025
@github-actions
Copy link
Contributor

This PR targets the master branch but does not come from dev or a hotfix/* branch.

Automatically setting the base branch to dev.

@github-actions github-actions bot changed the base branch from master to dev December 30, 2025 15:05
@AutoGPT-Agent
Copy link

Thank you for fixing these typos in the codebase!

I noticed there's a change in use-get-flag.ts that switches the CHAT feature flag from true to false. This appears to be a functional change rather than a typo fix, and it's not mentioned in your PR description.

Could you please:

  1. Either remove the feature flag change if it was unintentional
  2. Or update your PR title and description to accurately reflect this functional change if it was intentional

The actual typo fixes look good and are ready to merge once the feature flag issue is addressed.

@AutoGPT-Agent
Copy link

Thank you for fixing these typos in the codebase! The corrections to "separate", "noticeably", and "function" look good.

However, I noticed there's an unrelated change in src/services/feature-flags/use-get-flag.ts that changes the CHAT flag from true to false. This change isn't mentioned in your PR description and doesn't appear to be related to fixing typos.

Please either:

  1. Remove this unrelated change from your PR, or
  2. Update your PR description and title to reflect this additional change

Once that's addressed, this PR will be ready to merge. Thanks for your contribution!

@efeecllk
Copy link
Author

Thank you for fixing these typos in the codebase! The corrections to "separate", "noticeably", and "function" look good.

However, I noticed there's an unrelated change in src/services/feature-flags/use-get-flag.ts that changes the CHAT flag from true to false. This change isn't mentioned in your PR description and doesn't appear to be related to fixing typos.

Please either:

  1. Remove this unrelated change from your PR, or
  2. Update your PR description and title to reflect this additional change

Once that's addressed, this PR will be ready to merge. Thanks for your contribution!

I fixed unrelated change. sorry my darling.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicts Automatically applied to PRs with merge conflicts documentation Improvements or additions to documentation platform/backend AutoGPT Platform - Back end size/s

Projects

Status: 🆕 Needs initial review

Development

Successfully merging this pull request may close these issues.

4 participants