Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typescript 4 errors part 3 #1876

Merged

Conversation

celdrake
Copy link
Contributor

  • More Typescript v4 fixes.

Added a typed equivalent to Object.keys which returns a type based on the object.
This makes the casting when accessing the values unnecessary.

@celdrake celdrake requested review from a team as code owners January 18, 2023 16:14
@jkilzi
Copy link
Collaborator

jkilzi commented Jan 19, 2023

For the record, this thread seems to be relevant

@jkilzi
Copy link
Collaborator

jkilzi commented Jan 19, 2023

How can we enforce getKeys will be used instead of Object.keys?
Otherwise, we'll keep fixing this issue over and over again...

@celdrake
Copy link
Contributor Author

How can we enforce getKeys will be used instead of Object.keys? Otherwise, we'll keep fixing this issue over and over again...

With a quick search, I found this
https://github.com/remithomas/eslint-plugin-ban

Maybe it makes more sense to include it in the mono-repo instead of the master branch?

@celdrake celdrake merged commit cd07486 into openshift-assisted:master Jan 19, 2023
@celdrake celdrake deleted the fix-typescript-4-errors-part-3 branch January 19, 2023 13:01
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Jan 19, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Jan 19, 2023
* Fixing typescript errors in v4
* Async dispatch
celdrake pushed a commit to celdrake/assisted-ui-lib that referenced this pull request Jan 27, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Jan 29, 2023
* Fixing typescript errors in v4
* Async dispatch
celdrake pushed a commit to celdrake/assisted-ui-lib that referenced this pull request Jan 30, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Jan 30, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Jan 30, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Jan 30, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Jan 30, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Feb 2, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Feb 2, 2023
* Fixing typescript errors in v4
* Async dispatch
jkilzi added a commit to jkilzi/assisted-installer-ui that referenced this pull request Feb 19, 2023
* Fixing typescript errors in v4
* Async dispatch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants