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.

Sign up for GitHub

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

Jump to bottom

🔖 release v8.0.0 #609

Merged
merged 164 commits into from
May 9, 2024
Merged

🔖 release v8.0.0 #609

merged 164 commits into from
May 9, 2024

Conversation

techouse
Copy link
Collaborator

@techouse techouse commented May 3, 2024 •
edited
Loading

chopper

8.0.0

chopper_generator

8.0.0

chopper_built_value

3.0.0

  • Require Chopper ^8.0.0

yousinix and others added 30 commits May 25, 2021 09:26
@yousinix @JEuler
Fix Header Option Casting (#260)
f25fbe4 
Co-authored-by: Ivan Terekhin <i.terhin@gmail.com>
@JEuler
Fix for #259 (#263)
3d47735
@JEuler
4.0.1 fixes (#264)
8be965b
@luis901101
analyzer dependency upgraded (#296)
bcb0304
@rIIh
fix(generator): fix PartValueFile value not nullable if arg is (#288) (…
4e4607e 
…#293)
@JEuler
Chopper generator release 4.0.2 (#297)
939a83f
@rIIh
fix: fix this.body cast of null value when response body is null (#291)…
ff0ec87 
… (#292)
@JEuler
Interpolation fixes (#275)
ffcd945
@JEuler
encodeQueryComponent now encodeComponent (#278)
82b951f
@JEuler
Prevent double call on token refreshment (#276)
f7255b7
@JEuler
Fixes for #309 #308 (#310)
ddefa94
@michalsrutek
Remove new keyword from interceptors.md (#312)
0386c74
@JEuler @stewemetal
Analyzer upgrade (#320)
d4dab0d 
Co-authored-by: István Juhos <stewemetal@gmail.com>
@pingbird
Add unnecessary_brace_in_string_interps to lint ignores (#317)
bbe2c7a
@vipw @JEuler
Extend pragma to quiet the linter (#318)
6bdbd63 
Co-authored-by: Ivan Terekhin <i.terhin@gmail.com>
@maxroehrl
Fix converter getting called twice if using an authenticator with a J…
4185d14 
…sonConverter on the request (#324)
@ipcjs
migrate example to nullsafety (#331)
dbf4272
@ibadin
Resolve problem in main_json_serializable example (#328)
976d457
@meysam1717 @mysmartapply
Add @FiledMap @PartMap @PartFileMap (#335)
cc2da20 
Co-authored-by: Meysam Karimi <mysmartapply.it4@gmail.com>
@JEuler
Upgrade of analyzer (#340)
659b9f8
@techouse
Fix nullable QueryMap fails to compile (#344)
e167ba6
@techouse
Change return type of decodeJson to FutureOr in order to be able to s…
f9009ce 
…upport compute() (#345)
@techouse
Migrate from pedantic to lints ^2.0.0 with lints/recommended.yaml (#349)
5f2eb82
@JEuler
Version bumped for release (#352)
02cf2e0
@techouse
Revert analyzer to ^4.1.0 and silence linters for Element.enclosingEl…
34b3bda 
…ement (#354)
@techouse
[chopper_generator] Update analyzer to ^4.4.0 and code_builde to ^4.3…
426a16e 
….0 and migrate deprecated code (#358)
@techouse
Add Makefiles to streamline development (#357)
35fdf8f
@techouse
Add Bug Report Github issue template (#359)
5a59ebf
@techouse
[chopper_generator] Add types to the generated variables (#360)
fcf3478
@techouse
Provide an example using an Isolate Worker Pool with Squadron (#361)
6b3b840
@techouse techouse added the release prod release label May 3, 2024
@techouse
Copy link
Collaborator Author

techouse commented May 3, 2024 •
edited
Loading

@Guldem I have written a short migration guide around the Interceptors https://docs.google.com/document/d/1GkxktMbNi8VM5YzJwFuA19KMWeY96jrGmMsq867P28Y/edit?usp=sharing and plan to attach it to the changelog.

Would you mind checking it out and adding more info, notes and/or examples?

The publish dry run fails for the same reasons as in #606

CC / @JEuler

@codecov Codecov
Copy link

codecov bot commented May 3, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 95.45455% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 93.05%. Comparing base (ae505af) to head (e2b63e6).

Files Patch % Lines
...lib/src/interceptors/http_logging_interceptor.dart 95.45% 2 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #609      +/-   ##
==========================================
+ Coverage   92.85%   93.05%   +0.19%     
==========================================
  Files          22       22              
  Lines         490      504      +14     
==========================================
+ Hits          455      469      +14     
  Misses         35       35

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@techouse techouse changed the base branch from develop to master May 3, 2024 18:35
@Guldem
Copy link
Contributor

Guldem commented May 5, 2024

@Guldem I have written a short migration guide around the Interceptors https://docs.google.com/document/d/1GkxktMbNi8VM5YzJwFuA19KMWeY96jrGmMsq867P28Y/edit?usp=sharing and plan to attach it to the changelog.

Would you mind checking it out and adding more info, notes and/or examples?

The publish dry run fails for the same reasons as in #606

CC / @JEuler

I will check next Tuesday and add stuff if necessary 😄

@techouse
Copy link
Collaborator Author

techouse commented May 5, 2024 •
edited
Loading

I will check next Tuesday and add stuff if necessary 😄

Thanx man! Much appreciated!

The link from the comment

https://docs.google.com/document/d/1GkxktMbNi8VM5YzJwFuA19KMWeY96jrGmMsq867P28Y/edit?usp=sharing

is to a an editable version. The link in the changelog and PR description is an immutable copy of that same document.

@techouse techouse added this to the 8.0.0 milestone May 5, 2024
@Guldem
Copy link
Contributor

Guldem commented May 7, 2024

I will check next Tuesday and add stuff if necessary 😄

Thanx man! Much appreciated!

The link from the comment

https://docs.google.com/document/d/1GkxktMbNi8VM5YzJwFuA19KMWeY96jrGmMsq867P28Y/edit?usp=sharing

is to a an editable version. The link in the changelog and PR description is an immutable copy of that same document.

I added some feedback and updated the examples. Why not just have a markdown file in the repository with the migration plan instead of a google docs?

@techouse
Copy link
Collaborator Author

techouse commented May 7, 2024 •
edited
Loading

Why? I've seen this in a few other repos. So I thought why not do the same. 🤷‍♂️ It's probably down to the fact that the Google Doc can be updated outside of the repo.

@techouse techouse mentioned this pull request May 7, 2024
Merged
@techouse techouse marked this pull request as ready for review May 8, 2024 18:38
Guldem
Guldem approved these changes May 8, 2024
View reviewed changes
Copy link
Contributor

@Guldem Guldem left a comment

Choose a reason for hiding this comment

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

LGTM!

@techouse
Copy link
Collaborator Author

techouse commented May 8, 2024

LGTM!

Thanx for the feedback and awesome PR #547 ♥️

JEuler
JEuler approved these changes May 9, 2024
View reviewed changes
Copy link
Collaborator

@JEuler JEuler left a comment

Choose a reason for hiding this comment

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

Great job guys! Thank you so much for being active!

@techouse techouse merged commit df9801a into master May 9, 2024
10 of 12 checks passed
@techouse techouse deleted the release/v8.0.0 branch May 9, 2024 07:59
@Guldem
Copy link
Contributor

Guldem commented May 10, 2024

LGTM!

Thanx for the feedback and awesome PR #547 ♥️

Great job guys! Thank you so much for being active!

@techouse @JEuler Thanks for feedback and support!

@techouse techouse mentioned this pull request Jun 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Guldem Guldem Guldem approved these changes

@JEuler JEuler JEuler approved these changes

@lejard-h lejard-h Awaiting requested review from lejard-h lejard-h is a code owner

@meysam1717 meysam1717 Awaiting requested review from meysam1717 meysam1717 is a code owner

@stewemetal stewemetal Awaiting requested review from stewemetal stewemetal is a code owner

Assignees

@JEuler JEuler

Labels
release prod release
Projects
None yet
Milestone
8.0.0
Development

Successfully merging this pull request may close these issues.