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

Refactor: Enable TypeScript strict null checks #1221

Merged
merged 5 commits into from
Jan 23, 2017

Conversation

calebmer
Copy link
Contributor

@calebmer calebmer commented Jan 23, 2017

Enables TypeScript strictNullChecks for increased type safety guarantees. A longer term solution to #800

  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass
  • Update CHANGELOG.md with your change

@calebmer calebmer force-pushed the refactor/ts-strict-null-checks branch from 16a1c3b to 9555292 Compare January 23, 2017 17:00
Copy link
Contributor

@helfer helfer left a comment

Choose a reason for hiding this comment

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

Typings changes look good. Did it help catch any bugs?

@@ -11,7 +11,7 @@ Expect active development and potentially significant breaking changes in the `0
- Fix bug where there could be store inconsistencies for two dependent optimistic updates [PR #1144](https://github.com/apollostack/apollo-client/pull/1144)
- expose partial in ObservableQuery#currentResult [PR #1097](https://github.com/apollostack/apollo-client/pull/1097)
- Calculate `loading` from `networkStatus`. [PR #1202](https://github.com/apollostack/apollo-client/pull/1202)
- Fix typings error with `strictNullChecks` [PR #1188](https://github.com/apollostack/apollo-client/pull/1188)
- Fix typings error with `strictNullChecks` and enable TypeScript strict null checking in source code. [PR #1188](https://github.com/apollostack/apollo-client/pull/1188) and [PR #1221](https://github.com/apollostack/apollo-client/pull/1221)
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the rationale for putting this under 0.8 instead of vNEXT?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn’t realize there was a release and I missed the vNEXT when resolving the merge conflicts 😉

@helfer
Copy link
Contributor

helfer commented Jan 23, 2017

Also, we should merge this before we merged anything else, because otherwise we'll have a ton of conflicts.

@calebmer
Copy link
Contributor Author

@helfer
Copy link
Contributor

helfer commented Jan 23, 2017

Yeah, those don't look like super-likely candidates, but let's see.

@helfer helfer merged commit 6b6e8de into master Jan 23, 2017
@calebmer calebmer deleted the refactor/ts-strict-null-checks branch January 23, 2017 18:59
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants