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

Enable tsc --strict CI to prevent gaining more errors #23692

Merged
merged 1 commit into from
Nov 4, 2022

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Nov 3, 2022


This change is marked as an internal change (Task), so will not be included in the changelog.

@t3chguy t3chguy added the T-Task Tasks for the team like planning label Nov 3, 2022
@t3chguy t3chguy requested a review from a team as a code owner November 3, 2022 16:08
@t3chguy t3chguy merged commit 77d7598 into develop Nov 4, 2022
@t3chguy t3chguy deleted the t3chguy-patch-1 branch November 4, 2022 09:12
clarkf added a commit to clarkf/matrix-react-sdk that referenced this pull request Jan 23, 2023
This gets RoomView.ts to pass `tsc --strict`. Should be functionally unchanged.

A number of other files were touched to correct signatures.

See:
* element-hq/element-web#23542
* element-hq/element-web#23692

Signed-off-by: Clark Fischer <[email protected]>
clarkf added a commit to clarkf/matrix-react-sdk that referenced this pull request Jan 23, 2023
This gets RoomView.ts to pass `tsc --strict`. Should be functionally unchanged.

A number of other files were touched to correct signatures.

See:
* element-hq/element-web#23542
* element-hq/element-web#23692

Signed-off-by: Clark Fischer <[email protected]>
clarkf added a commit to clarkf/matrix-react-sdk that referenced this pull request Jan 24, 2023
This gets RoomView.ts to pass `tsc --strict`. Should be functionally unchanged.

A number of other files were touched to correct signatures.

See:
* element-hq/element-web#23542
* element-hq/element-web#23692

Signed-off-by: Clark Fischer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Tasks for the team like planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants