Skip to content

Merge pull request #666 from DiamondLightSource/pre-release/2023-R4.1 #389

Merge pull request #666 from DiamondLightSource/pre-release/2023-R4.1

Merge pull request #666 from DiamondLightSource/pre-release/2023-R4.1 #389

Triggered via pull request September 19, 2023 09:25
Status Success
Total duration 2m 40s
Artifacts

ci.yml

on: pull_request
Checkout, build, test and lint PHP code
2m 9s
Checkout, build, test and lint PHP code
JavaScript build, test and lint
2m 31s
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