Skip to content

Conversation

@lume-code
Copy link
Contributor

@lume-code lume-code commented Oct 3, 2025

Upgraded various dependencies to their latest versions:

  • Updated Dart SDK constraint from ">=3.8.0 <4.0.0" to ">=3.9.0 <4.0.0".
  • Adjusted build dependency range to ">=4.0.0 <5.0.0".
  • Updated gql_code_builder from version ^0.13.1 to ^0.15.1.
  • Updated build_runner from version ^2.0.3 to ^2.9.0.
  • Updated graphql from version ^5.2.0-beta.3 to ^5.2.2.

However, some tests fail. I’m not sure, but from on my understanding, it’s likely due to a formatting mismatch in more recent versions of gql_code_builder (double \n vs single \n).

Related to #404

Thanks @budde377

- Upgraded various dependencies to their latest versions for improved performance and compatibility.
- Updated Dart SDK constraint from ">=3.8.0 <4.0.0" to ">=3.9.0 <4.0.0".
- Adjusted build dependency range to ">=4.0.0 <5.0.0".
- Updated gql_code_builder from version ^0.13.1 to ^0.15.1.
- Updated build_runner from version ^2.0.3 to ^2.9.0.
- Updated graphql from version ^5.2.0-beta.3 to ^5.2.2.
Copilot AI review requested due to automatic review settings October 3, 2025 01:46
@lume-code lume-code changed the title Upgrade build to '4.0.0' Upgrade build to 4.0.0 Oct 3, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades the build system and dependencies to version 4.0.0, focusing on improving compatibility and performance. The changes include updating the Dart SDK constraint and several key dependencies to their latest stable versions.

  • Updated build dependency from version 2.x to 4.x range
  • Upgraded gql_code_builder, build_runner, and graphql packages to newer versions
  • Updated Dart SDK minimum requirement from 3.8.0 to 3.9.0

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@joey-kb
Copy link

joey-kb commented Oct 8, 2025

@budde377 , please take a look to this PR. Thank you in advance.

@budde377
Copy link
Contributor

budde377 commented Oct 8, 2025

@lume-code Thanks for submitting this PR! As expected it looks like the changed formatting are breaking the tests.

Do you mind doing the following:

  1. Switch the commented out lines in this file https://github.com/heftapp/graphql_codegen/blob/main/packages/graphql_codegen/test/builder_test.dart#L104-L105 - i.e. make the test overwrite the test file rather than create a new .expected file
  2. Rerun tests
  3. Change the changed lines back
  4. Commit and push.

That should reformat all the tests.

@lume-code
Copy link
Contributor Author

@budde377 Thanks for the tip. Done, and the tests are now passing.

@budde377 budde377 merged commit 180efe4 into heftapp:main Oct 9, 2025
5 of 7 checks passed
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