Skip to content

Merge pull request #559 from DiamondLightSource/improvement/LIMS-452/… #321

Merge pull request #559 from DiamondLightSource/improvement/LIMS-452/…

Merge pull request #559 from DiamondLightSource/improvement/LIMS-452/… #321

Triggered via push August 16, 2023 16:06
Status Success
Total duration 3m 59s
Artifacts

ci.yml

on: push
Checkout, build, test and lint PHP code
2m 4s
Checkout, build, test and lint PHP code
JavaScript build, test and lint
3m 47s
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