Merge pull request #688 from DiamondLightSource/master #459
John-Holt-Tessellaci.yml
on: push
Checkout, build, test and lint PHP code
1m 47s
JavaScript build, test and lint
3m 3s
Annotations
14 errors and 9 warnings
|
InvalidGlobal:
src/Authentication/Type/OIDC.php#L15
src/Authentication/Type/OIDC.php:15:13: InvalidGlobal: Cannot use global scope here (see https://psalm.dev/046)
|
|
InvalidStaticInvocation:
src/Page/Download.php#L530
src/Page/Download.php:530:13: InvalidStaticInvocation: Method Symfony\Component\HttpFoundation\ResponseHeaderBag::makeDisposition is not static, but is called statically (see https://psalm.dev/014)
|
|
InvalidStaticInvocation:
src/Page/Download.php#L536
src/Page/Download.php:536:13: InvalidStaticInvocation: Method Symfony\Component\HttpFoundation\ResponseHeaderBag::makeDisposition is not static, but is called statically (see https://psalm.dev/014)
|
|
UndefinedFunction:
src/Page/Processing.php#L427
src/Page/Processing.php:427:25: UndefinedFunction: Function SynchWeb\Page\str_contains does not exist (see https://psalm.dev/021)
|
|
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/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/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
|