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

feat: strict typescript #2235

Merged
merged 60 commits into from
Mar 4, 2022
Merged

feat: strict typescript #2235

merged 60 commits into from
Mar 4, 2022

Conversation

brunozoric
Copy link
Contributor

@brunozoric brunozoric commented Feb 14, 2022

Changes

Set strict to tsconfig.build.json.
Upgrade to Typescript 4.5.5

Note that project works with Typescript 4.6.2, but currently eslint does not support 4.6.x

How Has This Been Tested?

Jest and manually.

Note

Moved @rmwc/base to packages/ui due to TS error in that lib.

@brunozoric brunozoric added this to the 5.24.0 milestone Feb 14, 2022
@brunozoric brunozoric self-assigned this Feb 14, 2022
@brunozoric brunozoric marked this pull request as ready for review February 21, 2022 06:47
@brunozoric brunozoric changed the title feat: strict null checks handler feat: strict null checks ts Mar 1, 2022
@brunozoric brunozoric modified the milestones: 5.24.0, 5.25.0 Mar 1, 2022
@brunozoric brunozoric changed the title feat: strict null checks ts feat: strict typescript Mar 1, 2022
# Conflicts:
#	packages/api-headless-cms/src/content/plugins/graphqlFields/object.ts
#	packages/app-file-manager/src/admin/views/FileManagerSettings.tsx
#	packages/app-headless-cms/src/admin/components/FieldEditor/EditFieldDialog.tsx
#	packages/app-headless-cms/src/admin/views/contentModels/NewContentModelDialog.tsx
Copy link
Contributor

@endymion1818 endymion1818 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm afraid I didn't get to the end of the PR before it refreshed but here's where I got to ... hope it's useful

@brunozoric brunozoric merged commit c7c49e7 into next Mar 4, 2022
@Pavel910 Pavel910 deleted the feat/ts/strict-null-checks branch June 6, 2022 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants