Skip to content

Add rule for pinning PECL packages#1069

Open
phansys wants to merge 1 commit into
hadolint:masterfrom
phansys:issue_937
Open

Add rule for pinning PECL packages#1069
phansys wants to merge 1 commit into
hadolint:masterfrom
phansys:issue_937

Conversation

@phansys
Copy link
Copy Markdown

@phansys phansys commented Jan 19, 2025

Partially addresses #937, for the PECL package manager.

What I did

I've added rule DL3062 for pinning PECL packages.

How I did it

I've copied and edited the DL3016 rule, which does the same for NPM packages.

How to verify it

Instructions like these should produce a rule violation:

pecl install redis
pecl install redis-6.1.0 xdebug
pecl install pecl.php.net/redis

To Do

  • Add the spec;
  • Confirm all the allowed variants for the PECL channel definitions;
  • Update the docs;
  • Confirm the rule is working, as I have no local setup for the project yet.

@phansys phansys force-pushed the issue_937 branch 3 times, most recently from 57d9d99 to a00e174 Compare January 19, 2025 22:02
@phansys phansys marked this pull request as ready for review January 19, 2025 22:07
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.

1 participant