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 null pointer exception during the generation queries which return… #685

Merged

Conversation

dkaera
Copy link
Collaborator

@dkaera dkaera commented Aug 4, 2022

Recreated PR because the previous one was mistakenly updated during the solving conflicts.

@codecov-commenter
Copy link

codecov-commenter commented Aug 4, 2022

Codecov Report

Merging #685 (e1c9a6e) into develop (3dbe42f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #685   +/-   ##
========================================
  Coverage    90.18%   90.18%           
========================================
  Files           74       74           
  Lines         1874     1874           
========================================
  Hits          1690     1690           
  Misses         184      184           
Flag Coverage Δ
floor 90.57% <ø> (ø)
floor_generator 90.13% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
floor_generator/lib/misc/type_utils.dart 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@dkaera dkaera enabled auto-merge (squash) August 4, 2022 12:29
mqus
mqus previously approved these changes Aug 4, 2022
example/pubspec.lock Outdated Show resolved Hide resolved
ThomasMiddel
ThomasMiddel previously approved these changes Aug 17, 2022
@dkaera dkaera dismissed stale reviews from ThomasMiddel and mqus via e1c9a6e August 17, 2022 12:49
@ThomasMiddel ThomasMiddel merged commit 4e3562d into pinchbv:develop Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants