The Jenkins job translatewiki-shelllint just does:
find bin -maxdepth 1 -type f -print0 | xargs -t -n1 -0 bash -n
The repo already has a job that runs composer test so I guess we can migrate the behavior inside composer as a script that is local to the repo instead of hardcoded in a Jenkins job. That will let us remove the job.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| lint bash script from composer test command | translatewiki | master | +38 -1 | |
| Remove translatewiki-shelllint | integration/config | master | +7 -23 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | hashar | T119138 [keyresult] Migrate majority of CI jobs to Nodepool (part 2) | |||
| Duplicate | None | T136249 Track what tests we have left to convert to nodepool | |||
| Resolved | hashar | T119140 [keyresult] Migrate as many misc CI jobs as possible to Nodepool | |||
| Resolved | hashar | T160394 Replace job translatewiki-shelllint and have its functionality invoked from composer |
Change 342590 had a related patch set uploaded (by Hashar):
[translatewiki] lint bash script from composer test command
Change 342597 had a related patch set uploaded (by Hashar):
[integration/config] Remove translatewiki-shelllint
Change 342597 merged by jenkins-bot:
[integration/config] Remove translatewiki-shelllint
The job is no more in CI. Pending https://gerrit.wikimedia.org/r/#/c/342590/ which replaces the behavior by a PHP script invoked from composer test.
Change 342590 merged by jenkins-bot:
[translatewiki] lint bash script from composer test command