Skip to content

Apply PHPCS to the build tools and remove unused md5/sha1 hashes#34290

Merged
HLeithner merged 2 commits intojoomla:stagingfrom
zero-24:phpcs_build
Jun 1, 2021
Merged

Apply PHPCS to the build tools and remove unused md5/sha1 hashes#34290
HLeithner merged 2 commits intojoomla:stagingfrom
zero-24:phpcs_build

Conversation

@zero-24
Copy link
Copy Markdown
Contributor

@zero-24 zero-24 commented May 30, 2021

Summary of Changes

Apply PHPCS to the build tools and remove unused md5/sha1 hashes

Testing Instructions

  • make sure the builds (below this page) still works.
  • code review

Actual result BEFORE applying this Pull Request

Some phpcs issues as well as the generation of the not used md5 & sha1 hashes

Expected result AFTER applying this Pull Request

Lesser phpcs issues as well as no md5 nor sha1 hashes anymore

Documentation Changes Required

none

@PhilETaylor

This comment was marked as abuse.

* phpcs

* revert to suitable lang level
@richard67
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 8958b6c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34290.

1 similar comment
@Quy
Copy link
Copy Markdown
Contributor

Quy commented May 31, 2021

I have tested this item ✅ successfully on 8958b6c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34290.

@Quy
Copy link
Copy Markdown
Contributor

Quy commented May 31, 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34290.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 31, 2021
@zero-24 zero-24 added this to the Joomla! 3.9.28 milestone May 31, 2021
@HLeithner HLeithner merged commit 82321be into joomla:staging Jun 1, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jun 1, 2021
@zero-24 zero-24 deleted the phpcs_build branch June 1, 2021 16:54
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.

6 participants