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

Clarify package usage #79

Open
wants to merge 1 commit into
base: latest
Choose a base branch
from

Conversation

pixelbrackets
Copy link

@pixelbrackets pixelbrackets commented Aug 13, 2021

Split installation and usage description.

Extend description how the package works and when it runs.

Explain need for Composer 2 in manual version checks (see #66)

Split installation and usage description.

Extend description how the package works and when it runs.

Explain need for Composer 2 in manual version checks (see Roave#66)

Refs Roave#66

You may manually trigger a version check only by using the `--dry-run` flag on an update.

This requires Composer version 2 however, due to a wont-fix issue in deprecated Composer 1.x.
Copy link
Member

Choose a reason for hiding this comment

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

The tool also works with composer:^1 - there are limitations with it, but it has worked since forever with it?

Copy link
Author

Choose a reason for hiding this comment

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

See linked issue #66, the manual check with existing composer.lock does not work with Composer 1.x. It needs Composer 2.

The other commands (compose require and composer update without dry-run) work fine with Composer 1. But this section is about the dry-run.

PS: The diff is a hard to to read → this is what the rendered version of the text change looks like:
https://github.com/pixelbrackets/SecurityAdvisories/blob/20210813_add-composer-version-requirement/README.md

The checks are only executed when adding a new dependency via `composer require` or when running `composer update`:
deploying an application with a valid `composer.lock` and via `composer install` won't trigger any security versions
checking.
### Manual checks
Copy link
Member

Choose a reason for hiding this comment

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

Would rather say that these are for continuous integration

Copy link
Author

Choose a reason for hiding this comment

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

What do you mean with "these". Replace the headline "Manual checks" with "Continuous integration"?

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.

None yet

2 participants