flake8
passed.
JSHint
passed.
Convert BaseCollection and ResourceCollection to spina.
Review Request #13004 — Created May 3, 2023 and submitted
This change ports our basic collection types over to spina and
TypeScript.
Ran js-tests.
| Summary | ID |
|---|---|
| 5519b2c6fe5d456dd709389e487dd55d78c4faf3 |
| Description | From | Last Updated |
|---|---|---|
|
> on the next line. |
 chipx86
|
|
|
This is missing a docstring. |
chipx86
|
|
|
Alphabetical order. |
chipx86
|
|
|
Missing a docstring. |
chipx86
|
|
|
Elsewhere for wrapping, I've been putting extends on the next line. I feel it helps with readability a bit. |
chipx86
|
|
|
Alphabetical order. |
chipx86
|
|
|
> on the next line. |
chipx86
|
|
|
Can we make this an interface, for easier reference by subclasses? |
chipx86
|
|
|
Should use unknown where possible. |
chipx86
|
|
|
As in other changes, ideally unit tests should just import the necessary modules themselves. |
chipx86
|
-
-
-
-
-
-
Elsewhere for wrapping, I've been putting
extendson the next line. I feel it helps with readability a bit. -
-
-
-
-
reviewboard/static/rb/js/common/resources/collections/tests/resourceCollectionTests.ts (Diff revision 1) As in other changes, ideally unit tests should just import the necessary modules themselves.
Commits: |
|
|||||||
|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+568 -324) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.