Skip to content

LIMS-940: Add PR template #317

LIMS-940: Add PR template

LIMS-940: Add PR template #317

Triggered via pull request August 15, 2023 15:51
Status Success
Total duration 3m 27s
Artifacts

ci.yml

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

Annotations

10 errors and 9 warnings
JavaScript build, test and lint: client/src/js/models/protein.js#L19
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/models/visit.js#L54
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/models/visit.js#L55
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/models/visit.js#L56
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/models/visit.js#L56
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/models/visit.js#L56
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/models/visit.js#L57
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/modules/cell/views/beamlines.js#L45
Move function declaration to function body root
JavaScript build, test and lint: client/src/js/modules/dc/models/autointegration.js#L16
Do not access attributes directly. Use set() and get() instead
JavaScript build, test and lint: client/src/js/modules/dc/models/autointegration.js#L19
Do not access attributes directly. Use set() and get() instead
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