Intervention / Intervention/image
Rename parse() methods to fromString()
Open
Nobody has claimed this yet.
major-release
- Dominant language
- PHP
- Stars
- 14.4k
- Forks
- 1.5k
- Avg merge
- 17h 51m
- Merged PRs (30d)
- 11
Description
The following classes have a static parse() method which is the equivalent for creating objects from string. I'm considering renaming this method to fromString() since there are other from methods in the library.
- Intervention\Image\DataUri
- Intervention\Image\Color
- Intervention\Image\Colors\Oklch\Color
- Intervention\Image\Colors\Oklab\Color
- Intervention\Image\Colors\Cmyk\Color
- Intervention\Image\Colors\Hsv\Color
- Intervention\Image\Colors\Rgb\Color
- Intervention\Image\Colors\Hsl\Color
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Review the static parse() methods in DataUri, Color, and the listed color classes, then search the repository for their usages. Determine the complete rename scope and check how existing callers and tests refer to these methods. Done means the naming is consistent across all listed classes and their known usages.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- computer-graphics
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100