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

Null safety (non stable version) #219

Merged
merged 53 commits into from
Mar 27, 2021
Merged

Null safety (non stable version) #219

merged 53 commits into from
Mar 27, 2021

Conversation

lejard-h
Copy link
Owner

No description provided.

JEuler and others added 30 commits April 10, 2020 12:33
Runtime base url note update

Update faq.md
* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>
@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>
Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml
Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter
grahamsmith and others added 10 commits February 11, 2021 07:43
* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>
* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>
* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>
Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.
* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>
* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>
@lejard-h lejard-h requested review from stewemetal and JEuler March 22, 2021 17:35
Copy link
Collaborator

@stewemetal stewemetal left a comment

Choose a reason for hiding this comment

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

All of us (all contributors) did a really good job on this. 🥳

I found only a few small things in the new additions.

@lejard-h Will the release update the gitbook documentation automatically?

chopper/CHANGELOG.md Outdated Show resolved Hide resolved
chopper_generator/CHANGELOG.md Outdated Show resolved Hide resolved
chopper_generator/pubspec.yaml Show resolved Hide resolved
repository: https://github.com/lejard-h/chopper
author: Hadrien Lejard <[email protected]>

environment:
sdk: ">=2.1.0 <3.0.0"
sdk: ">=2.12.0 <3.0.0"

Choose a reason for hiding this comment

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

I think this is okay for the final version 4.0.0, but shouldn't this be ">=2.12.0-0 <3.0.0" during the 4.0.0-nullsafety.x development versions so people can already use this in projects which are not migrated to the latest stable yet?

Copy link
Owner Author

Choose a reason for hiding this comment

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

I can change that yes

Copy link
Owner Author

Choose a reason for hiding this comment

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

@jpelgrim unfortunately I need to do a lot dependency changes to support unstable sdk (source_gen, build, http ...)
We will support stable 2.12 only sorry :/

Copy link

@jpelgrim jpelgrim Mar 23, 2021

Choose a reason for hiding this comment

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

Okay, understand. Maybe rename the PR then? And maybe go for version 4.0.0 directly? If there's a 4.0.0-nullsafety.x version people might expect it to be a non-stable null safe version which they can use in non-null safe projects.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@jpelgrim has a point here. I know we discussed in an earlier thread that we want to test this release out in the wild before saying that it is stable, but considering the above, we just might "lie" to our users ("unstable" release with stable dependencies and requirements).

Copy link
Owner Author

Choose a reason for hiding this comment

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

I don't think it's a problem, using 4.0.0-nullsafety.x just mean the package itself is unstable for any reasons (partial migration, untested ...) it does not mean we use unstable sdk or unstable dependencies (it could but not here).

If the naming is the issue, we could just rename it to 4.0.0-dev.x

Copy link
Owner Author

Choose a reason for hiding this comment

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

I just want to avoid the release of a stable 4.0.0 then introduce breaking change a few weeks later because of an issue on null safety.

Copy link
Contributor

Choose a reason for hiding this comment

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

4.0.0-nullsafety.x is a really good well-known approach

Choose a reason for hiding this comment

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

Yeah, I think so too. The argument was

Unfortunately I need to do a lot dependency changes to support unstable sdk (source_gen, build, http ...)

But I think that needs to be done for the final null-safe version anyway. So if we just depend on the latest <package>-nullsafety.x version for all the dependencies in the chopper package and depend on Dart SDK ">=2.12.0-0 <3.0.0" is a good approach to support a set of true 4.0.0-nullsafety.x versions people can play around with.

allcontributors bot and others added 5 commits March 24, 2021 10:05
* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>
* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>
* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>
* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>
* update Changelog

* contributors

* cleanup
@lejard-h lejard-h requested a review from stewemetal March 25, 2021 15:05
Copy link
Collaborator

@stewemetal stewemetal left a comment

Choose a reason for hiding this comment

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

Everything seems to be in order except one little thing that actually I screwed up a few days ago in one of my suggestions.
I'll approve the PR so you guys won't have to wait for me any longer, but please fix that one line in the changelog. 🙏

@@ -1,5 +1,14 @@
# Changelog

## 4.0.0-nullsafety.0

- Sound null safety support
Copy link
Collaborator

Choose a reason for hiding this comment

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

My bad, we should just go for "Null safety support" here if we're doing 4.0.0-nullsafety.0.

Suggested change
- Sound null safety support
- Null safety support

@lejard-h lejard-h merged commit 480e0c2 into master Mar 27, 2021
lejard-h added a commit that referenced this pull request Apr 3, 2021
* coverage

* coverage test

* push

* regenerate

* regenerate

* token

* fix

* fix

* fix fix fix

* dart action

* cleanup

* Null safety (non stable version) (#219)

* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* cleanup

* regenerate

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
lejard-h added a commit that referenced this pull request Apr 10, 2021
* Null safety (non stable version) (#219)

* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* fix params and headers

* fix

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
lejard-h added a commit that referenced this pull request Apr 13, 2021
* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

* flutter fav badge

* Fix Flutter favorites badge (#234)

* chopper_built_value - null safety support (#231)

* Fix typo in converters.md (#236)

* Cleanup deprecated (#228)

* remove deprecated filefield

* Request.replace

* PartValue.replace

* Response.replace

* cleanup

* add chopper_built_value to github actions (#232)

* Cleanup nullable hashmap APIs (#233)

* GIthub actions - code coverage (#229)

* coverage

* coverage test

* push

* regenerate

* regenerate

* token

* fix

* fix

* fix fix fix

* dart action

* cleanup

* Null safety (non stable version) (#219)

* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* cleanup

* regenerate

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* Fix infinite loop when using Authenticators (#241)

* Generator - fix headers and required parameters (#239)

* Null safety (non stable version) (#219)

* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* fix params and headers

* fix

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* 4.0.0-nullsafety.1

* chopper_built_value 1.0.0-nullsafety.0

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
lejard-h added a commit that referenced this pull request May 17, 2021
* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

* flutter fav badge

* Fix Flutter favorites badge (#234)

* chopper_built_value - null safety support (#231)

* Fix typo in converters.md (#236)

* Cleanup deprecated (#228)

* remove deprecated filefield

* Request.replace

* PartValue.replace

* Response.replace

* cleanup

* add chopper_built_value to github actions (#232)

* Cleanup nullable hashmap APIs (#233)

* GIthub actions - code coverage (#229)

* coverage

* coverage test

* push

* regenerate

* regenerate

* token

* fix

* fix

* fix fix fix

* dart action

* cleanup

* Null safety (non stable version) (#219)

* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* cleanup

* regenerate

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* Fix infinite loop when using Authenticators (#241)

* Generator - fix headers and required parameters (#239)

* Null safety (non stable version) (#219)

* GZip note

* Update faq.md

* CR fixes

* Moved to the faq.md

* HTTP Auth example

* CR fix

* CR fix

* Base url note

Runtime base url note update

Update faq.md

* Packages upgraded (#142)

* Release 3.0.3

* add develop branch to travis

* fix travis badge

* Make code quality improvements in generator.dart (#147)

* Add optionalBody parameter to suppress warnings (#151)

* Add optionalBody parameter to suppress warnings

Fixes #149

* Improve formatting

* Remove redundant _methodWithBody function

Use a single source of truth (the method's optionalBody property)

* Show optional body hint in warning message

* Make optionalBody configurable for all methods

Explicitly set the default value for every method

* Specify type for optionalBody argument

Co-authored-by: Wilko Manger <[email protected]>

* Fix typo and invalid code sample in documentation (#156)

* Preserve uri current query params in buildUri (#166)

* This allows chopper API methods with generic attributes (#170)

@post(path: URL.DOCUMENT_PRIOR_INFO)
  Future<Response<T>> getPriorInfo<T>(@Body() Params params);

Which is usefull when single API method may respond differently
depending on request parameter values.

Co-authored-by: Aleksandr Malkov <[email protected]>

* Make conversion helpers can be `async` (#175)

#174 
- locally tested

* Dependency update (#178)

Version bumped

Removed Angular / Web examples

With nullability added

Update generator.dart

Dependency cleanup

Update pubspec.yaml

Update pubspec.yaml

* Re-add Angular example and fix travis (#181)

Re add example removed here (https://github.com/lejard-h/chopper/pull/178/files)

Upgrade to Angular 6, but I had to remove jaguar_serializer which is not maintain anymore (last update 1 year ago).

Then use new Dart command to run pub, analyzer and formatter

* Publish package from travis (#182)

* fix

* fix scopes

* allow_failure

* comment override

* Add suport for JsonApi utf8 serialization (#185)

* Add suport for JsonApi utf8 serialization

* Move json api header into a constant

* The Big Public API Documentation update, volume 1. (#189)

* Update converters.md with wording and formatting fixes

* Update the documentation of classes and functions in interceptor.dart

* Update the documentation of classes and functions in base.dart

* Update the documentation of public functions in utils.dart

* Update the documentation of public classes and functions in request.dart

* Update the documentation of public classes and functions in response.dart

* Update the documentation of ConvertRequest, ConvertResponse, and FactoryConverter in annotations.dart

* Fix typos in the documentation in annotations.dart

* Remove TODO from ChopperService.definitionType's documentation

* Add example usage to FactoryConverter's documentation

* Fix accidental typo in ChopperService's class name

* The Big Public API Documentation update, volume 2. (#193)

* Update the documentation of BuiltValueConverter

* Update the documentation and usage guide of BuiltValueConverter

* Update description texts in pubspec.yaml files

* Change homepage tag to documentation in chopper/pubspec.yaml

* Change homepage tag to documentation in chopper_built_value/pubspec.yaml and make it point to its actual documentation page

* Change homepage tag to documentation in chopper_generator/pubspec.yaml

* Change homepage tag to documentation in chopper/example/pubspec.yaml and make it point to the actual documentation

* Fix wording in chopper_generator/README.md

* Fix wording in chopper_built_value/README.md

* Fix HTML escape sequences escaping into markdown and inline docs; Fix code sample syntax errors in some docs

Those pesky escapists.

* Add dynamic version tag to built-value-converter.md

* Fix typo in the package name in the installation section of built-value-converter.md

* Fix typo in built-value-converter.md

* Reorganize and update chopper/README.md

* The Big Public API Documentation Update, volume 3. (#196)

* Rework and update getting-started.md

* Reword parts of the API docs in annotations.dart

* Add description of path resolution behavior in requests.md
This commit addresses issue #195

* Reword parts of the documentations in requests.md

* Reword the error handling description in getting-started.md

* Reword and update most of the documentation in requests.md

* Add missing @s to annotation mentions in getting-started.md

* Remove an unnecessary new line from getting-started.md

* Add further explanation to an example service's create method in getting-started.md

* Update the README of chopper_built_value

* Update the README of chopper_generator

* Update and reword top level README.md and chopper/README.md

* Try to fix text formatting in requests.md

* Fix wording requests.md's form URL encoded section

* Remove unsafe hint on build_runner from getting-started.md

* Implement authenticator (#198)

* Implement authenticator

* Changes for PR

Co-authored-by: Ivan Terekhin <[email protected]>

* Add Flutter Favorite badge (#206)

* feature/updated doc (#214)

* updated doc

* fixed typo

Co-authored-by: dafinrs <[email protected]>

* Feature/support body get (#201)

* Feature/null safety migration (#212)

* Pubspec for ChopperBuiltValue

* Begin resolving null safety dependencies

* Initial change

* Implement null safety on chopper_generator

* Fix comments on PR

* Fixed some comments

* version updates

* fixed issues with versions

* fix comments on chopperClient nullability

* fixed tests

* Fixed issues with nullable client

* Bumped build version to 2.0.0

* updated libs

* made parts not nullable

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper/README.md

Co-authored-by: Ivan Terekhin <[email protected]>

* Update chopper_generator/pubspec.yaml

Co-authored-by: Ivan Terekhin <[email protected]>

* Fix for null safety in applyHeaders

* More deps upgrade

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: István Juhos <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: uladzimir_paliukhovich <>

* Add double quotes to Curl print String (#218)

Prints the Curl URL wrapped with double-quotes.
It fixes issues when pasting the resulting log to the command line, where the curl request in some cases would not be closed by the terminal or fail to execute.

* Enable GitHub action (#216)

* github acions with mono_repo

* fail test

* cleanup

* badge

* regenerate

* regenerate

* format

* fix analyzer

* fix analyzer

* fix test

* regenerate

* cleanup badge

* Naming fix

* remove dev chanel

* ignore vscode

* revert readme change

* remove travis

* format

* fix

* fix

* format

* fix tests

Co-authored-by: Ivan Terekhin <[email protected]>

* Add an OPTIONS request type (#215)

* Add an OPTIONS request type

* Null safety fix

Co-authored-by: Ivan Terekhin <[email protected]>

* changelog

* docs: add lejard-h as a contributor (#221)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* docs: add stewemetal as a contributor (#222)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add JEuler as a contributor (#223)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add fryette as a contributor (#224)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* docs: add Vovanella95 as a contributor (#225)

* docs: update README.md [skip ci]

* docs: create .all-contributorsrc [skip ci]

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hadrien Lejard <[email protected]>

* update Changelog for 4.0.0 release (#220)

* update Changelog

* contributors

* cleanup

* changelog

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* fix params and headers

* fix

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* 4.0.0-nullsafety.1

* chopper_built_value 1.0.0-nullsafety.0

* Authenticator exported in chopper.dart (#247)

* Generator upgrade (#248)

* Update pubspec.yaml

* More packages upgraded

* Generator null safety fix

* Null safety stable (#253)

* github actions publish workflow

* rename

* comment before publish

* comment overrides

Co-authored-by: Ivan Terekhin <[email protected]>
Co-authored-by: Juhos István <[email protected]>
Co-authored-by: Louis Matthijssen <[email protected]>
Co-authored-by: Wilko Manger <[email protected]>
Co-authored-by: Tamas Balogh <[email protected]>
Co-authored-by: Mohammad Omidvar Tehrani <[email protected]>
Co-authored-by: cpthooch <[email protected]>
Co-authored-by: Aleksandr Malkov <[email protected]>
Co-authored-by: Study-Log <[email protected]>
Co-authored-by: Daniel Gomez <[email protected]>
Co-authored-by: Graham Smith <[email protected]>
Co-authored-by: Eugeny Sampir <[email protected]>
Co-authored-by: dafi <[email protected]>
Co-authored-by: dafinrs <[email protected]>
Co-authored-by: Alfredo Bautista <[email protected]>
Co-authored-by: Uladzimir Paliukhovich <[email protected]>
Co-authored-by: Uladzimir_Paliukhovich <[email protected]>
Co-authored-by: Alex Queudot <[email protected]>
Co-authored-by: Shane Farmer <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@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.