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

🔀 merge updated workflows to master #426

Merged
merged 61 commits into from
May 15, 2023

Conversation

techouse
Copy link
Collaborator

@techouse techouse commented May 15, 2023

The moment of truth.

  • coverage reporting should run ✅
  • dart pub publish --dry-run should run in the PR ✅
  • dart pub publish should run when merged ✅

yousinix and others added 30 commits May 25, 2021 09:26
Co-authored-by: Ivan Terekhin <i.terhin@gmail.com>
Co-authored-by: István Juhos <stewemetal@gmail.com>
Co-authored-by: Ivan Terekhin <i.terhin@gmail.com>
Co-authored-by: Meysam Karimi <mysmartapply.it4@gmail.com>
techouse and others added 18 commits December 11, 2022 15:56
Co-authored-by: Ivan Terekhin <231950+JEuler@users.noreply.github.com>
* Extracted the http logging interceptor
Make http logging interceptor configurable (none, basic, headers, body)

* Minor tweaks
Updated documentation

* Simplified onRequest body and startMessage logic.
Made onResponse more consistent with onRequest.
* Made applyHeaders field name case insensitive.
Added tests for applyHeaders

* fixed quotes lint check
* #290 implement Equatable

* ➕ add faker and data_fixture_dart as dev dependencies

* ✅ add Request equatability test

* ✅ add Response equatability test

* 🔥 Remove some obsolete equatables

* 🔥 Remove some obsolete equatables

* ✅ add PartValue equatability test
* ⬆️ Bump Dart constraint to ">=2.17.0 <3.0.0"
* ⬆️ Bump mono_repo Yaml
* 🚨 Ignore Element.enclosingElement3 warning
# Conflicts:
#	.github/workflows/dart.yml
#	.github/workflows/publish.yml
#	.github/workflows/publish_dry_run.yml
#	mono_repo.yaml
#	tool/ci.sh
@codecov
Copy link

codecov bot commented May 15, 2023

Codecov Report

Merging #426 (a8be0f6) into master (96c17ef) will increase coverage by 4.17%.
The diff coverage is 95.09%.

@@            Coverage Diff             @@
##           master     #426      +/-   ##
==========================================
+ Coverage   89.45%   93.63%   +4.17%     
==========================================
  Files           5        8       +3     
  Lines         332      440     +108     
==========================================
+ Hits          297      412     +115     
+ Misses         35       28       -7     
Impacted Files Coverage Δ
chopper/lib/src/interceptor.dart 73.33% <71.42%> (-7.16%) ⬇️
chopper/lib/src/response.dart 88.88% <88.88%> (+22.22%) ⬆️
chopper/lib/src/base.dart 94.73% <94.73%> (+1.08%) ⬆️
chopper/lib/src/http_logging_interceptor.dart 95.91% <95.91%> (ø)
chopper/lib/src/extensions.dart 100.00% <100.00%> (ø)
chopper/lib/src/request.dart 100.00% <100.00%> (+7.50%) ⬆️
chopper/lib/src/utils.dart 100.00% <100.00%> (ø)
chopper_built_value/lib/chopper_built_value.dart 96.66% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@techouse techouse merged commit 636dbc3 into master May 15, 2023
@techouse techouse deleted the chore/merge-updated-workflows-to-master branch May 15, 2023 06:05
@techouse
Copy link
Collaborator Author

Works like a charm 😎

@techouse techouse added the sync Branch sync label Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sync Branch sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.