Drop the I in some interface names after reviewing with Ira

[lyonsil]

This change is 

[katherinejensen00]

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @lyonsil)

[irahopkinson]

I have one non-blocking question since it's out of scope for this PR, so ready to merge.

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lyonsil)

---

src/shared/utils/util.ts line 152 at r1 (raw file):

 * This type is basically a pointer to an object.
 */
export interface Container<T> {

I'm not sure this type belongs in this util.ts file. Having said that I'm not sure where else it might go so this is non-blocking.

[katherinejensen00]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lyonsil)

[lyonsil]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @lyonsil)

---

src/shared/utils/util.ts line 152 at r1 (raw file):

Previously, irahopkinson (Ira Hopkinson) wrote…

I'm not sure this type belongs in this util.ts file. Having said that I'm not sure where else it might go so this is non-blocking.

I couldn't find a better place to put it, and TJ didn't have another suggestion, so that's why it went where it did. I can move it into it's own file if you think that's better.

[irahopkinson]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @lyonsil)