Skip to content

Update build dependency to >=v3 <5.0.0 - #216

Merged
shamblett merged 3 commits into
shamblett:issue215from
oskarwilliams:update-build-dependency
May 6, 2026
Merged

shamblett merged 3 commits into
shamblett:issue215from
oskarwilliams:update-build-dependency

Conversation

@oskarwilliams

Copy link
Copy Markdown
Contributor

Updates build dependency to v3 and higher.
Adds testing of config file generation.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the package’s build dependency constraint to allow newer major versions and adds automated tests to validate CoAP config YAML → Dart generation, including aligning the example config with the builder’s required version key.

Changes:

  • Broaden build (and related dev) dependency constraints and add build_test for builder testing.
  • Add a comprehensive configBuilder test suite covering type handling, nested keys, special-casing, and error behavior.
  • Update the example config YAML/Dart to include the required version entry and ensure generated output matches.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
test/coap_builder_test.dart Adds unit tests for the config builder, including an example-file roundtrip check.
pubspec.yaml Updates build/build_runner constraints and adds build_test.
example/config/coap_config.yaml Adds required version key to satisfy builder validation.
example/config/coap_config.dart Updates generated example output to include version and DTLS-related getters.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread test/coap_builder_test.dart Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@shamblett
shamblett changed the base branch from master to issue215 May 6, 2026 13:33
@shamblett
shamblett merged commit b69d024 into shamblett:issue215 May 6, 2026
1 of 4 checks passed
shamblett added a commit that referenced this pull request May 6, 2026
* Update build dependency to >=v3 <5.0.0 (#216)

* Allow for more up to date versions of build

* Add testing values

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Oskar Williams <oskar@tewke.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Formatting

---------

Co-authored-by: Oskar Williams <me@oskarw.com>
Co-authored-by: Oskar Williams <oskar@tewke.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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