search

nhost / nhost-dart

Nhost Dart & Flutter packages
https://nhost.io
MIT License
90 stars 32 forks source link

chore(release): Publish packages #141

Closed onehassan closed 2 months ago

onehassan commented 2 months ago

User description

Package Name Current Version Updated Version Update Reason
nhost_dart 2.0.4 2.0.5 updated with patch changes
nhost_sdk 5.3.0 5.3.1 updated with patch changes
nhost_flutter_auth 4.0.4 4.0.5 dependency was updated
nhost_flutter_graphql 3.0.5 3.0.6 dependency was updated
nhost_graphql_adapter 4.0.4 4.0.5 dependency was updated
nhost_storage_dart 2.0.4 2.0.5 dependency was updated
nhost_gql_links 4.0.5 4.0.6 dependency was updated
nhost_auth_dart 2.2.0 2.2.1 dependency was updated
nhost_functions_dart 2.0.4 2.0.5 dependency was updated

PR Type

Enhancement, Bug fix

Description

Changes walkthrough 📝

Relevant files
Documentation
10 files
CHANGELOG.md
Update CHANGELOG for new release versions                               
CHANGELOG.md
  • Added new release notes for multiple packages.
  • Documented fixes and dependency updates.
    		•  +45/-0   
    CHANGELOG.md
    Update nhost_auth_dart CHANGELOG for v2.2.1                           
    packages/nhost_auth_dart/CHANGELOG.md - Added entry for version 2.2.1. - Documented dependency update.
    		+4/-0     
    CHANGELOG.md
    Update nhost_dart CHANGELOG for v2.0.5                                     
    packages/nhost_dart/CHANGELOG.md
  • Added entry for version 2.0.5.
  • Documented fix for test parameter.
    		•  +4/-0     
    CHANGELOG.md
    Update nhost_flutter_auth CHANGELOG for v4.0.5                     
    packages/nhost_flutter_auth/CHANGELOG.md - Added entry for version 4.0.5. - Documented dependency update.
    		+4/-0     
    CHANGELOG.md
    Update nhost_flutter_graphql CHANGELOG for v3.0.6               
    packages/nhost_flutter_graphql/CHANGELOG.md - Added entry for version 3.0.6. - Documented dependency update.
    		+4/-0     
    CHANGELOG.md
    Update nhost_functions_dart CHANGELOG for v2.0.5                 
    packages/nhost_functions_dart/CHANGELOG.md - Added entry for version 2.0.5. - Documented dependency update.
    		+4/-0     
    CHANGELOG.md
    Update nhost_gql_links CHANGELOG for v4.0.6                           
    packages/nhost_gql_links/CHANGELOG.md - Added entry for version 4.0.6. - Documented dependency update.
    		+4/-0     
    CHANGELOG.md
    Update nhost_graphql_adapter CHANGELOG for v4.0.5               
    packages/nhost_graphql_adapter/CHANGELOG.md - Added entry for version 4.0.5. - Documented dependency update.
    		+4/-0     
    CHANGELOG.md
    Update nhost_sdk CHANGELOG for v5.3.1                                       
    packages/nhost_sdk/CHANGELOG.md
  • Added entry for version 5.3.1.
  • Documented fix for handling null phoneNumber.
    		•  +4/-0     
    CHANGELOG.md
    Update nhost_storage_dart CHANGELOG for v2.0.5                     
    packages/nhost_storage_dart/CHANGELOG.md - Added entry for version 2.0.5. - Documented dependency update.
    		+4/-0     
    Dependencies
    11 files
    pubspec.yaml
    Bump nhost_auth_dart version and dependencies                       
    packages/nhost_auth_dart/pubspec.yaml
  • Bumped version to 2.2.1.
  • Updated nhost_sdk dependency to 5.3.1.
    		•  +2/-2     
    pubspec.yaml
    Bump nhost_dart version and dependencies                                 
    packages/nhost_dart/pubspec.yaml
  • Bumped version to 2.0.5.
  • Updated multiple dependencies to latest versions.
    		•  +7/-7     
    pubspec.yaml
    Bump nhost_flutter_auth version and dependencies                 
    packages/nhost_flutter_auth/pubspec.yaml
  • Bumped version to 4.0.5.
  • Updated nhost_dart dependency to 2.0.5.
    		•  +2/-2     
    pubspec.yaml
    Bump nhost_flutter_graphql version and dependencies           
    packages/nhost_flutter_graphql/pubspec.yaml
  • Bumped version to 3.0.6.
  • Updated nhost_flutter_auth dependency to 4.0.5.
    		•  +2/-2     
    pubspec.yaml
    Bump nhost_functions_dart version and dependencies             
    packages/nhost_functions_dart/pubspec.yaml
  • Bumped version to 2.0.5.
  • Updated nhost_sdk dependency to 5.3.1.
    		•  +2/-2     
    pubspec.yaml
    Bump nhost_gql_links version and dependencies                       
    packages/nhost_gql_links/pubspec.yaml
  • Bumped version to 4.0.6.
  • Updated nhost_sdk dependency to 5.3.1.
    		•  +2/-2     
    pubspec.yaml
    Update nhost_graphql_adapter example dependencies               
    packages/nhost_graphql_adapter/example/pubspec.yaml - Updated `nhost_dart` dependency to 2.0.5.
    		+1/-1     
    pubspec.yaml
    Bump nhost_graphql_adapter version and dependencies           
    packages/nhost_graphql_adapter/pubspec.yaml
  • Bumped version to 4.0.5.
  • Updated multiple dependencies to latest versions.
    		•  +3/-3     
    pubspec.yaml
    Bump nhost_sdk version                                                                     
    packages/nhost_sdk/pubspec.yaml - Bumped version to 5.3.1.
    		+1/-1     
    pubspec.yaml
    Update nhost_storage_dart example dependencies                     
    packages/nhost_storage_dart/example/pubspec.yaml - Updated `nhost_auth_dart` dependency to 2.2.1.
    		+1/-1     
    pubspec.yaml
    Bump nhost_storage_dart version and dependencies                 
    packages/nhost_storage_dart/pubspec.yaml
  • Bumped version to 2.0.5.
  • Updated nhost_sdk dependency to 5.3.1.
    		•  +2/-2     

    💡 PR-Agent usage: Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    codiumai-pr-agent-pro[bot] commented 2 months ago

    PR Reviewer Guide 🔍

    (Review updated until commit https://github.com/nhost/nhost-dart/commit/c81556960df76c9935d8d64e41ed1a3b11b83804)

    ⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Key issues to review

    Redundant Information
    The packages listed under "Packages with dependency updates only" are repeated in the "Packages with other changes" section. This redundancy could be removed for clarity and to avoid potential confusion.
    codiumai-pr-agent-pro[bot] commented 2 months ago

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Possible bug
    Add null checks to boolean casts to prevent potential null pointer exceptions ___ **Consider adding null checks for emailVerified and phoneNumberVerified when casting
    from JSON to ensure robustness against potential null values in the JSON data.** [packages/nhost_sdk/lib/src/api/auth_api_types.dart [139-141]](https://github.com/nhost/nhost-dart/pull/141/files#diff-a611d97d65069a8f508b41d5dced2816a657086b86cfa99be35e282a79cd7a62R139-R141) ```diff -emailVerified: json['emailVerified'] as bool, -phoneNumberVerified: json['phoneNumberVerified'] as bool, +emailVerified: json['emailVerified'] as bool? ?? false, +phoneNumberVerified: json['phoneNumberVerified'] as bool? ?? false, ``` - [ ] **Apply this suggestion**
    Suggestion importance[1-10]: 9 Why: Adding null checks for `emailVerified` and `phoneNumberVerified` is crucial for preventing potential null pointer exceptions, which enhances the robustness of the code.
    		9
    Best practice
    Ensure type safety by using Object instead of dynamic for the equality operator ___ **Replace the dynamic type with Object in the equality operator to ensure type safety
    and consistency with Dart best practices.** [packages/nhost_flutter_auth/lib/src/provider.dart [55]](https://github.com/nhost/nhost-dart/pull/141/files#diff-f97a430716e7193059ada18ed2b8c2e5edbcbd3ee7a4113da3838036a9131d78R55-R55) ```diff +bool operator ==(Object other) { - ``` - [ ] **Apply this suggestion**
    Suggestion importance[1-10]: 8 Why: Using `Object` instead of `dynamic` for the equality operator improves type safety and aligns with Dart best practices, making the code more reliable.
    		8
    Improve type safety by specifying a more precise type for the metadata map ___ **Use more specific types for metadata in the constructor to ensure type safety and
    avoid potential runtime errors.** [packages/nhost_sdk/lib/src/api/auth_api_types.dart [118]](https://github.com/nhost/nhost-dart/pull/141/files#diff-a611d97d65069a8f508b41d5dced2816a657086b86cfa99be35e282a79cd7a62R118-R118) ```diff -final Map? metadata; +final Map? metadata; ``` - [ ] **Apply this suggestion**
    Suggestion importance[1-10]: 7 Why: Specifying a more precise type for the `metadata` map improves type safety and reduces the risk of runtime errors, although it is a minor enhancement.
    		7
    Enhancement
    Use specific types or validation for phone numbers to ensure data integrity ___ **Consider using a more specific type for phoneNumber to ensure that it adheres to
    expected formats and enhances validation.** [packages/nhost_dart/test/test_helpers.dart [29]](https://github.com/nhost/nhost-dart/pull/141/files#diff-0c7fc89faad9702cf5ce74197c216100b1cf262d470fe5b25218349be0e14f7cR29-R29) ```diff -phoneNumber: defaultTestPhone, +phoneNumber: defaultTestPhone, // Ensure defaultTestPhone is a valid phone number format ``` - [ ] **Apply this suggestion**
    Suggestion importance[1-10]: 6 Why: Adding a comment to ensure `defaultTestPhone` is a valid phone number format is a minor enhancement that can help maintain data integrity, but it doesn't enforce validation directly.
    		6