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 CS8602 Warnings in Weather.razor (BlazorApp) – Handle Nullable forecasts and user.Identity #3266

Merged
merged 2 commits into from
Feb 26, 2025

Conversation

gladjohn
Copy link
Contributor

@gladjohn gladjohn commented Feb 26, 2025

Fix CS8602 Warnings in Weather.razor (BlazorApp) – Handle Nullable forecasts and user.Identity

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Description

This pull request includes changes to the Weather.razor component in the Blazor application to handle potential null values and initialize arrays properly.

Improvements and code safety enhancements:

Fixes #3265

@gladjohn gladjohn requested a review from a team as a code owner February 26, 2025 01:26
Copy link

Summary

Summary
Generated on: 2/26/2025 - 1:40:54 AM
Coverage date: 2/26/2025 - 1:40:05 AM - 2/26/2025 - 1:40:51 AM
Parser: MultiReport (3x Cobertura)
Assemblies: 0
Classes: 0
Files: 0
Line coverage:
Covered lines: 0
Uncovered lines: 0
Coverable lines: 0
Total lines: 0
Covered branches: 0
Total branches: 0
Method coverage: Feature is only available for sponsors

Coverage

No assemblies have been covered.

@gladjohn gladjohn merged commit ee18453 into master Feb 26, 2025
4 checks passed
@gladjohn gladjohn deleted the gladjohn/idweb-fix-warn branch February 26, 2025 04:20
Copy link

Summary

Summary
Generated on: 2/26/2025 - 4:34:11 AM
Coverage date: 2/26/2025 - 4:33:21 AM - 2/26/2025 - 4:34:08 AM
Parser: MultiReport (3x Cobertura)
Assemblies: 0
Classes: 0
Files: 0
Line coverage:
Covered lines: 0
Uncovered lines: 0
Coverable lines: 0
Total lines: 0
Covered branches: 0
Total branches: 0
Method coverage: Feature is only available for sponsors

Coverage

No assemblies have been covered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CS8602 Warning: Dereference of a Possibly Null Reference in Weather.razor (BlazorApp)
4 participants