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.

Sign up for GitHub

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

Jump to bottom

[INFRA] Activate alwaysStrict compiler option #458

Merged
merged 1 commit into from
Aug 5, 2020
Merged

[INFRA] Activate alwaysStrict compiler option #458

merged 1 commit into from
Aug 5, 2020

Conversation

csouchet
Copy link
Member

@csouchet csouchet commented Aug 4, 2020 •
edited by tbouffard
Loading

Covers #421

@csouchet csouchet added chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first labels Aug 4, 2020
This was referenced Aug 4, 2020
Merged
Merged
@csouchet csouchet force-pushed the 421-alwaysStrict branch 2 times, most recently from f916ede to 5d1c334 Compare August 4, 2020 17:16
@csouchet csouchet marked this pull request as ready for review August 4, 2020 17:16
@tbouffard tbouffard changed the title [INFRA] Activate TypeScript strict type checking options: alwaysStrict [INFRA] Activate alwaysStrict compiler options Aug 4, 2020
@tbouffard tbouffard changed the title [INFRA] Activate alwaysStrict compiler options [INFRA] Activate alwaysStrict compiler option Aug 4, 2020
@csouchet csouchet removed the depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first label Aug 5, 2020
@csouchet csouchet requested a review from tbouffard August 5, 2020 07:40
@tbouffard tbouffard merged commit 17c8652 into master Aug 5, 2020
@tbouffard tbouffard deleted the 421-alwaysStrict branch August 5, 2020 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tbouffard tbouffard tbouffard approved these changes

Assignees
No one assigned
Labels
chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@csouchet @tbouffard