Skip to content

phpcs updates #77

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.

Sign up for GitHub

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

Jump to bottom
Merged
rkoopmans merged 29 commits into from
Jan 27, 2026
Merged

phpcs updates #77

rkoopmans merged 29 commits into from
Jan 27, 2026

Conversation

@tijmenbruggeman
Copy link
Collaborator

@tijmenbruggeman tijmenbruggeman commented Nov 24, 2025 •
edited
Loading

Updates and additions to formatting dependencies. This PR will also help ensure the plug-in stays compatible with php5.6.

Changes

  • Update license to GPL-2.0-or-later. License "GPL-2.0+" is a deprecated SPDX license identifier, use "GPL-2.0-or-later" instead
  • updated phpcs and wordpress codestyles, ran autoformatter
  • manual fixes
    • line endings
    • use gmdate instead of date
    • explicit visibility for methods
    • allow one class per file
    • strict in_array comparisons for types
    • ignore strict comparison check (===) as this would functionally change the workings and I don't want to add functional changes in this PR.

@tijmenbruggeman
License "GPL-2.0+" is a deprecated SPDX license identifier, use "GPL-…
8e9a414 
…2.0-or-later" instead
@tijmenbruggeman
phpcompatibility/php-compatibility
a9b8fd9
@tijmenbruggeman
Update formatting dependencies
d7e7c54
@tijmenbruggeman
autoformat
dd3b033
@tijmenbruggeman
fix translations: after formatting
ca7d4f8
@tijmenbruggeman
replace date with gmdate
33b6b12
@tijmenbruggeman
chore: resolve line endings
04f0af5
@tijmenbruggeman
chore: make constructor public
cf46547
@tijmenbruggeman
chore: resolve issues with translations and line endings
4b9c5fc
@tijmenbruggeman
chore: merge command and cli into one class
5bcd0fd
@tijmenbruggeman
Split tiny-picture up in sources
bc75feb
@tijmenbruggeman
chore: fix strict comparisons in_array
60d0f15
@tijmenbruggeman
chore: resolve formatting for visibility explicits
7472a27
@tijmenbruggeman
Ignore prepared
03fefe8
@tijmenbruggeman
Fix unit tests
726da41
@tijmenbruggeman
Import extraced classes
bf518d5
@tijmenbruggeman
require once
1f74c47
@tijmenbruggeman
Use __DIR__
e17016d
@tijmenbruggeman
Merge branch 'master' of github.com:wcreateweb/wordpress-plugin into …
97d7e4f 
…phpcs-compat
@tijmenbruggeman
Remove phpcompatibility/phpcompatibility-wp
9b8a05b
@tijmenbruggeman
Lint after merge
6ebd526
@tijmenbruggeman
Ignore linelength
137b2b0
@tijmenbruggeman
Remove base class after merge conflict
8e5488b
@tijmenbruggeman
Remove PHPCompat from formatting
42b2449
@tijmenbruggeman
fix: esc value in html as img urls could have xss vulnerability
eab1e19
@tijmenbruggeman
chore: use array_merge instead of foreach append for consistancy
57f2600
@tijmenbruggeman
Merge branch 'master' of github.com:wcreateweb/wordpress-plugin into …
f9ca458 
…phpcs-compat
@tijmenbruggeman
autoformat
a9bcbb7
@tijmenbruggeman
add mock esc_attr
f9576e9
@rkoopmans rkoopmans merged commit ecfdfea into tinify:master Jan 27, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rkoopmans rkoopmans rkoopmans approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tijmenbruggeman @rkoopmans