Skip to content

Conversation

@miquelbeltran
Copy link
Contributor

@miquelbeltran miquelbeltran commented Dec 17, 2024

ci: fix analyze issue

Description 📝

  • Purpose: Fixes CI analyze error
  • Approach: fix switch statement and pull dependencies
    • Note: This does not upgrade major dependencies, just minor/patch versions.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Updates

  • Improve connectivity switch-case.
  • Pull minor/patch dependencies via flutter pub get

Test plan 🧪

-Unit testing and flutter analyze report no errors

Author to check 👓

  • Project and all contained modules builds successfully
  • Self-/dev-tested
  • Unit/UI/Automation/Integration tests provided where applicable
  • Code is written to standards
  • Appropriate documentation written (code comments, internal docs)

Reviewer to check ✔️

  • Project and all contained modules builds successfully
  • Change has been dev-/reviewer-tested, where possible
  • Unit/UI/Automation/Integration tests provided where applicable
  • Code is written to standards
  • Appropriate documentation written (code comments, internal docs)

Copy link
Contributor

@TheRealAgentK TheRealAgentK left a comment

Choose a reason for hiding this comment

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

LGTM

@TheRealAgentK TheRealAgentK merged commit dd8e29a into develop Dec 19, 2024
11 checks passed
@miquelbeltran miquelbeltran deleted the fix-ci branch December 19, 2024 20:14
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.

3 participants