Skip to content

fix: adds manual platform entries to pubspec.yaml#17

Merged
itsjavi merged 1 commit into
mainfrom
fix/platform-detection
Jun 16, 2026
Merged

fix: adds manual platform entries to pubspec.yaml#17
itsjavi merged 1 commit into
mainfrom
fix/platform-detection

Conversation

@pandeymangg

Copy link
Copy Markdown
Contributor

adds manual platform entries android and ios to the pubspec.yaml file in the formbricks package since the auto detection only detects android on pub.dev

@pandeymangg pandeymangg requested a review from itsjavi June 16, 2026 05:28
@sonarqubecloud

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: fe26bdcb-9e79-4612-8077-e81c9eb3b31e

📥 Commits

Reviewing files that changed from the base of the PR and between 57e735d and 2629868.

📒 Files selected for processing (1)
  • packages/formbricks/pubspec.yaml

Walkthrough

The pubspec.yaml for the formbricks Flutter package gains an explicit platforms section that declares android and ios as supported platforms. This block is inserted before the existing dependencies section. No other metadata, environment constraints, dependencies, or dev dependencies are modified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding manual platform entries to pubspec.yaml for Android and iOS platforms.
Description check ✅ Passed The description is directly related to the changeset, explaining why the platform entries are being added and what problem it solves.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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


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.

@itsjavi itsjavi merged commit a4b11c6 into main Jun 16, 2026
10 checks passed
@itsjavi itsjavi deleted the fix/platform-detection branch June 16, 2026 05:52
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.

2 participants