Skip to content

Add missing php85-tokenizer extension#530

Merged
frenck merged 1 commit into
mainfrom
frenck-2026-0284
Apr 12, 2026
Merged

Add missing php85-tokenizer extension#530
frenck merged 1 commit into
mainfrom
frenck-2026-0284

Conversation

@frenck

@frenck frenck commented Apr 12, 2026

Copy link
Copy Markdown
Member

Proposed Changes

Adds the php85-tokenizer PHP extension to the runtime dependencies and removes the --ignore-platform-req=ext-tokenizer workaround from the composer install command.

Grocy performs its own PHP extension checks at startup and fails to start without tokenizer. The composer flag was suppressing the build-time check but the runtime check was never satisfied.

Related Issues

(Github link to related issues or pull requests)

@frenck frenck added the bugfix Inconsistencies or issues which will cause a problem for users or implementors. label Apr 12, 2026
@coderabbitai

coderabbitai Bot commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@frenck has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 2 minutes and 11 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 2 minutes and 11 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc0d9ac2-cd0c-40f3-9ee0-d0fd04f0c20a

📥 Commits

Reviewing files that changed from the base of the PR and between a16ebe8 and cb2d35e.

📒 Files selected for processing (1)
  • grocy/Dockerfile
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch frenck-2026-0284

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.

@frenck frenck merged commit 46b164a into main Apr 12, 2026
15 checks passed
@frenck frenck deleted the frenck-2026-0284 branch April 12, 2026 10:38
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix Inconsistencies or issues which will cause a problem for users or implementors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant