Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

Fix all @ts-expect-error TS() FIXME: errors #258

Open
mtfranchetto opened this issue Aug 18, 2022 · 0 comments
Open

Fix all @ts-expect-error TS() FIXME: errors #258

mtfranchetto opened this issue Aug 18, 2022 · 0 comments
Labels
tech-debt Legacy issues or new tools adoption

Comments

@mtfranchetto
Copy link
Contributor

Description

After we have migrated to Typescript using Codemods #188, in order to have a working build, the tool has left several FIXME comments to suppress Typescript errors.
Our goal is to remove all the errors by typing correctly the affected files, and have better typings across the codebase.

The list of comments to address can be found with a simple search:
https://github.com/aws-samples/pcluster-manager/search?q=FIXME
@mtfranchetto mtfranchetto added the tech-debt Legacy issues or new tools adoption label Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
tech-debt Legacy issues or new tools adoption
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mtfranchetto