Skip to content

Removed open ai api key environment variable#64

Merged
AkhileshNegi merged 1 commit intomainfrom
feat/env-var-update
Feb 24, 2026
Merged

Removed open ai api key environment variable#64
AkhileshNegi merged 1 commit intomainfrom
feat/env-var-update

Conversation

@rkritika1508
Copy link
Collaborator

@rkritika1508 rkritika1508 commented Feb 24, 2026

Summary

Target issue is #65.
Removed the Open AI API Key environment variable since it's not being used anywhere so far. It will be taken up in the future list of validators, so, we will add it then itself.

Checklist

Before submitting a pull request, please ensure that you mark these task.

  • Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
  • If you've fixed a bug or added code that is tested and has test cases.

Notes

  • Removed OPENAI_API_KEY configuration requirement from environment setup examples.

@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2f7bd27 and b5de4a2.

📒 Files selected for processing (2)
  • .env.example
  • .env.test.example
💤 Files with no reviewable changes (2)
  • .env.test.example
  • .env.example

📝 Walkthrough

Walkthrough

Removed OPENAI_API_KEY environment variable configuration and its associated comment from two example environment files (.env.example and .env.test.example). No functional code changes or runtime behavior modifications.

Changes

Cohort / File(s) Summary
Environment Configuration Examples
.env.example, .env.test.example
Removed OPENAI_API_KEY configuration line and preceding comment about doc transformation from both files.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

ready-for-review

Suggested reviewers

  • nishika26

Poem

🐰 The example files hop so light,
No API keys in sight,
Cleaned up and neat,
The change complete!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title directly and accurately describes the main change: removing the OpenAI API key environment variable from example files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/env-var-update

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.

@rkritika1508 rkritika1508 marked this pull request as ready for review February 24, 2026 07:53
@AkhileshNegi AkhileshNegi merged commit 09931fb into main Feb 24, 2026
2 checks passed
@AkhileshNegi AkhileshNegi deleted the feat/env-var-update branch February 24, 2026 08:17
@AkhileshNegi AkhileshNegi added the enhancement New feature or request label Feb 24, 2026
@rkritika1508 rkritika1508 linked an issue Feb 24, 2026 that may be closed by this pull request
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.

Removed redundant environment variables- Open AI API key

2 participants