Skip to content

Conversation

@jobenjada
Copy link
Member

code is self-explanatory :)

@jobenjada jobenjada requested a review from pandeymangg October 6, 2025 15:31
@coderabbitai
Copy link

coderabbitai bot commented Oct 6, 2025

Walkthrough

Adds apps/playground/.env.example containing NEXT_PUBLIC_FORMBRICKS_API_HOST=http://localhost:3000 and NEXT_PUBLIC_FORMBRICKS_ENVIRONMENT_ID=YOUR-ENVIRONMENT-ID. Updates apps/playground/.gitignore to stop ignoring the example file (allowing !.env.example). Overhauls apps/playground/README.md: replaces "Getting Started" with "Setup", adds pnpm install, instructs creating env.local with the two Formbricks variables, uses pnpm dev to start, and documents SDK methods (setUserId, setAttribute, setAttributes, setEmail, setLanguage, track, logout). No runtime logic changes.

Pre-merge checks

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description Check ❓ Inconclusive The description “code is self-explanatory :)” is too vague and does not summarize any of the changes such as adding an example .env file or updating the README, so it fails to convey meaningful information about the pull request. Please update the description to briefly explain the additions (the new .env.example file, .gitignore update, and README setup instructions).
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly summarizes the main changes by indicating the addition of an example environment file for easier setup and improvements to the README, which matches the core modifications in this pull request.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 496dbd7 and 7a417db.

📒 Files selected for processing (1)
  • apps/playground/.gitignore (1 hunks)
🔇 Additional comments (1)
apps/playground/.gitignore (1)

35-35: Thanks for restoring the safe ignore pattern.

.env* continues to shield real env files while the negation keeps the example tracked—exactly what we needed.


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 3

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2ee2052 and 496dbd7.

📒 Files selected for processing (3)
  • apps/playground/.env.example (1 hunks)
  • apps/playground/.gitignore (1 hunks)
  • apps/playground/README.md (1 hunks)
🧰 Additional context used
🪛 dotenv-linter (3.3.0)
apps/playground/.env.example

[warning] 2-2: [EndingBlankLine] No blank line at the end of the file

(EndingBlankLine)

🪛 markdownlint-cli2 (0.18.1)
apps/playground/README.md

13-13: Ordered list item prefix
Expected: 1; Actual: 2; Style: 1/1/1

(MD029, ol-prefix)


22-22: Ordered list item prefix
Expected: 1; Actual: 3; Style: 1/1/1

(MD029, ol-prefix)

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 7, 2025

Copy link
Contributor

@pandeymangg pandeymangg left a comment

Choose a reason for hiding this comment

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

LGTM! thanks! 🙌

@pandeymangg pandeymangg added this pull request to the merge queue Oct 7, 2025
Merged via the queue into main with commit 0e3be08 Oct 7, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants