Skip to content

feat: add .env.sample template for environment configuration#17

Merged
srtaalej merged 7 commits intoai-appsfrom
ale-ai-apps-envsample
Oct 6, 2025
Merged

feat: add .env.sample template for environment configuration#17
srtaalej merged 7 commits intoai-appsfrom
ale-ai-apps-envsample

Conversation

@srtaalej
Copy link
Copy Markdown
Contributor

@srtaalej srtaalej commented Oct 3, 2025

Type of change

  • New feature
  • Bug fix
  • Documentation

Summary

adds .env.sample template, python-dotenv dependency for loading .env files, and updates app.py to load environment variables from .env file

Requirements

  • I have ensured the changes I am contributing align with existing patterns and have tested and linted my code
  • I've read and agree to the Code of Conduct

@srtaalej srtaalej requested review from mwbrooks and zimeg October 3, 2025 17:29
@srtaalej srtaalej added the enhancement New feature or request label Oct 3, 2025
Copy link
Copy Markdown
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

@srtaalej Thanks a bunch for putting together this PR!

🧪 While testing, I noticed that we're not loading the .env in the source code. I'll push up a commit to do that.

Comment thread .env.sample Outdated
Comment thread README.md
@mwbrooks
Copy link
Copy Markdown
Member

mwbrooks commented Oct 3, 2025

Commit 0133ca9 now loads the .env when the app is being initialized.

Copy link
Copy Markdown
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

✅ Looks good on my side, it's nice to see the .env file support land!

🧪 Manually tested by cloning this branch locally, using a custom .env and running python3 app.py. I didn't use the Slack CLI in this testing, so it looks good to me!

Comment thread app.py
Copy link
Copy Markdown
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

@srtaalej @mwbrooks LGTM! I like that this file makes clear which variables ought be set to run the app.

Some comments below note thoughts I had in review alongside some changes made, but if that all seems alright I think this is good to merge 🚢 💨

Comment thread .gitignore
Comment thread README.md
Comment thread requirements.txt
@srtaalej srtaalej merged commit 20419a8 into ai-apps Oct 6, 2025
3 checks passed
@srtaalej srtaalej deleted the ale-ai-apps-envsample branch October 6, 2025 20:42
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