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

Allow plugin robloach/component-installer, because it is required by components/handlebars.js #1262

Merged
merged 2 commits into from
Feb 17, 2022

Conversation

janvanmansum
Copy link
Contributor

@janvanmansum janvanmansum commented Jan 5, 2022

Reasons for creating this PR

When running composer install --no-dev I get a question about whether to trust the robloach/component-installer plug-in. In composer.json it is mentioned as a requirement for components/handlebars.js. robloach/component-installer is deprecated ("abandoned") but I know too little about PHP to solve this in a cleaner way. Hopefully, a new version of handlebars will use the new plug-in https://packagist.org/packages/oomphinc/composer-installers-extender.

The interactive part of the installation is incovenient because I am using Ansible to do the installation.

Link to relevant issue(s), if any

n/a

Description of the changes in this PR

  • Adds robloach/component-installer as an allowed plug-in to composer.json
  • Unrelated: adds /.idea to .gitignore as a convenience for people like myself who edit the project in IntelliJ.

Known problems or uncertainties in this PR

As far as I understand the only change that this PR applies is that the user who executes the composer install command is not longer asked whether to trust the robloach/component-installer plug-in. However, currently there is no real choice.

Checklist

  • phpUnit tests pass locally with my changes
    don't know how to run these, but I haven't changed any PHP code, so I would be suprised if they don't pass anymore.
  • I have added tests that prove my fix is effective or that my feature works (if not, explain why below)
    not sure how to do that
  • The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

…dependency (#1)

* Allow plugin robloach/component-installer, because it is required by components/handlebars.js

* Adding /.idea to .gitignore so project can be edited in IntelliJ

* Removed .idea files from PR

* newline at end of .gitignore
* Allow plugin robloach/component-installer, because it is required by components/handlebars.js

* Adding /.idea to .gitignore so project can be edited in IntelliJ

* Removed .idea files from PR

* newline at end of .gitignore

* Upgraded Twig

* Added composer.lock
@sonarcloud
Copy link

sonarcloud bot commented Feb 11, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@joelit
Copy link
Contributor

joelit commented Feb 17, 2022

Thanks for the PR! I will add this to the next release. :)

@joelit joelit merged commit 68449a2 into NatLibFi:v2.13-maintenance Feb 17, 2022
@joelit joelit mentioned this pull request Feb 17, 2022
@osma
Copy link
Member

osma commented Mar 1, 2022

This PR was merged to v2.13-maintenance branch instead of master. New PRs should always target master. This needs to be reapplied so it gets into the new release.

@joelit
Copy link
Contributor

joelit commented Mar 2, 2022

Created a new PR (#1280) from the commits of this one to be merged in the master branch.

@miguelvaara miguelvaara added this to the 2.14 milestone Mar 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

4 participants