Merge pull request #158 from oss-slu/issue-142-fix-rational-periodic-… #97
Annotations
7 warnings
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions/setup-node@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
lint:
Frontend/src/pages/ExploreSystems.js#L83
React Hook useEffect has missing dependencies: 'fetchFamilies', 'fetchFilteredSystems', 'setFilters', 'setPage', and 'systems?.length'. Either include them or remove the dependency array. You can also replace multiple useState variables with useReducer if 'setPagesDisplay' needs the current value of 'systems.length'
|
lint:
Frontend/src/pages/ExploreSystems.js#L88
React Hook useEffect has missing dependencies: 'fetchFamilies' and 'fetchFilteredSystems'. Either include them or remove the dependency array
|
lint:
Frontend/src/pages/ExploreSystems.js#L240
Expected '===' and instead saw '=='
|
lint:
Frontend/src/pages/ExploreSystems.js#L249
Unexpected string concatenation of literals
|
lint:
Frontend/src/pages/SystemDetails.js#L25
React Hook useEffect has a missing dependency: 'fetchDataForCSV'. Either include it or remove the dependency array
|