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

Draft: v0.2.0 #11

Merged
merged 29 commits into from
Nov 20, 2023
Merged

Draft: v0.2.0 #11

merged 29 commits into from
Nov 20, 2023

Conversation

GlenDC
Copy link
Member

@GlenDC GlenDC commented Nov 15, 2023

giant refactor: immutable services and impl traits

  • use 'impl Future' returns for trait contracts: prevents hidden requirments
  • refactor tower design around immutable services: easier to reason and use, more natural to use like this, makes it also more clear that user needs to handle state properly, prevents state keeping in service when not required and easier to integrate with other code bases in ecosystem (e.g. hyper)

just

Add justfile for easier developer experience (just qa to run them all).

- use 'impl Future' returns for trait contracts: prevents hidden requirments
- refactor tower design around immutable services: easier to reason and use,
  more natural to use like this, makes it also more clear that user needs to handle state properly,
  prevents state keeping in service when not required and easier to integrate
  with other code bases in ecosystem (e.g. hyper)
@GlenDC GlenDC marked this pull request as draft November 15, 2023 13:25
glendc added 28 commits November 15, 2023 22:10
getting stuck on trait bounds regarding ServiceBuilder...
hmmm
Next steps:

- complete 'tower_async_hyper':
   - integrate tests in tower-http;
   - add documentation;
- remove unneeded hyper references in tower-async-http (especially in doctests)
- reintroduce tower-async-bridge + make todo tests pass
- update changelogs
- review the http upgrade PR in tower-http + compare with tower-async-http PR (this one)
- use git version of tower-async in Rama to see if all is good to use
- do not use git versions of deps where not needed (eg hyper-util is now available)
instead of direct git references
now on_eos is called again, woohoo
@GlenDC GlenDC merged commit 347f34d into master Nov 20, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant