Skip to content
Build and Test SynchWeb

LIMS-1458: Fix GitHub warnings #934

Sign in to view logs

LIMS-1458: Fix GitHub warnings

LIMS-1458: Fix GitHub warnings #934

Triggered via pull request December 3, 2024 09:37
@ndg63276ndg63276
synchronize #841
improvement/LIMS-1458/fix-github-warnings
Status Success
Total duration 1m 39s
Artifacts –

ci.yml

on: pull_request
Checkout, build, test and lint PHP code
1m 16s
Checkout, build, test and lint PHP code
JavaScript build, test and lint
1m 29s
JavaScript build, test and lint
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 11 warnings
JavaScript build, test and lint: client/src/js/modules/types/mx/shipment/views/container-mixin.js#L391
Redundant double negation
JavaScript build, test and lint: client/src/js/modules/types/xpdf/shipment/views/sampletable.js#L195
'SampleTableView' is already defined
JavaScript build, test and lint: client/src/js/text.js#L206
Do not access Object.prototype method 'hasOwnProperty' from target object
JavaScript build, test and lint: client/src/js/text.js#L273
Do not access Object.prototype method 'hasOwnProperty' from target object
Checkout, build, test and lint PHP code
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
JavaScript build, test and lint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
JavaScript build, test and lint: client/src/js/modules/dc/views/queuebuilder.js#L27
Do not silence events
JavaScript build, test and lint: client/src/js/modules/dc/views/queuebuilder.js#L28
Do not silence events
JavaScript build, test and lint: client/src/js/modules/dc/views/queuebuilder.js#L158
Do not silence events
JavaScript build, test and lint: client/src/js/modules/shipment/collections/platetypes.js#L139
Do not silence events
JavaScript build, test and lint: client/src/js/views/search.js#L169
Use this.$ instead of $ in views
JavaScript build, test and lint: client/src/js/views/search.js#L170
Use this.$ instead of $ in views
JavaScript build, test and lint: client/src/js/views/search.js#L172
Use this.$ instead of $ in views
JavaScript build, test and lint: client/src/js/views/search.js#L185
Use this.$ instead of $ in views
JavaScript build, test and lint: client/src/js/views/search.js#L186
Use this.$ instead of $ in views