Page MenuHomeIn-Portal Phabricator

INP-1862 - Execute "lint" prior to deploy during the build
ClosedPublic

Authored by alex on May 30 2024, 7:16 AM.

Details

Test Plan
  1. in the IDE:
    • open modules/custom/units/sections/custom_eh.php file for editing
    • add a syntax error in there, e.g. replace function OnOverrideClonedSubItems($event) with function OnOverride ClonedSubItems($event)
    • save changes
  2. in CLI:
      • run the phing -f tools/build/build_custom.xml db-dump command to create a dump
    • for each of the build-*.sql files created in the system directory:
      • remove the mysqldump: [Warning] Using a password on the command line interface can be insecure. line (if found)
      • save changes
    • run the phing -f tools/build/build_custom.xml verify command
    • confirm, that build output clearly shows where the syntax error is made

Diff Detail

Repository
rINP In-Portal
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

alex created this revision.May 30 2024, 7:16 AM
alex requested review of this revision.May 30 2024, 7:16 AM
alex added a project: Restricted Project.May 30 2024, 7:17 AM
erik accepted this revision.May 30 2024, 10:35 AM
This revision is now accepted and ready to land.May 30 2024, 10:35 AM
This revision was automatically updated to reflect the committed changes.