Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(owlbot-php): install @prettier/plugin-php #1416

Closed
wants to merge 1 commit into from

Conversation

tmatsuo
Copy link
Contributor

@tmatsuo tmatsuo commented Apr 12, 2022

fixes #1415

@tmatsuo tmatsuo requested a review from a team April 12, 2022 17:20
@tmatsuo tmatsuo added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 12, 2022
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 12, 2022
@@ -39,6 +39,9 @@ ENV SYNTHTOOL_TEMPLATES="/synthtool/synthtool/gcp/templates"
ENV npm_config_cache=/tmp/.npm
RUN mkdir /tmp/.npm && chmod 1777 /tmp/.npm

# Install @prettier/plugin-php
RUN npm install --global prettier @prettier/plugin-php@^0.18.4
Copy link
Contributor

Choose a reason for hiding this comment

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

I might make this 0.8.4 rather than ^0.18.4.

Copy link
Contributor

Choose a reason for hiding this comment

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

(otherwise it's hard to know specifically what patch version we might have gotten).

@tmatsuo tmatsuo closed this Apr 12, 2022
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.

Post-processor container relies on external dependency
3 participants