Skip to content
This repository has been archived by the owner on Feb 25, 2022. It is now read-only.

upgrade to typesystem 0.3.0 #683

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

upgrade to typesystem 0.3.0 #683

wants to merge 2 commits into from

Conversation

aminalaee
Copy link
Member

@aminalaee aminalaee commented Sep 17, 2021 •
edited
Loading

Right now apistar has a loose dependency on typesystem.

This PR will support latest release of typesystem and also pin dependency.

@aminalaee aminalaee mentioned this pull request Sep 17, 2021
Merged
@aminalaee
Copy link
Member Author

@tomchristie Can you review this?

tomchristie
tomchristie approved these changes Sep 24, 2021
View reviewed changes
Copy link
Member

@tomchristie tomchristie left a comment

Choose a reason for hiding this comment

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

Sure thing

@aminalaee
Copy link
Member Author

@tomchristie I don't have write access here, feel free to merge this when ok.

@aminalaee
Copy link
Member Author

aminalaee commented Sep 25, 2021 •
edited
Loading

@tomchristie We also need a release.

@aminalaee aminalaee mentioned this pull request Oct 9, 2021
Closed
@waghanza
Copy link

@tomchristie It blocks responder (since apistar is on of it dependencies) to use the latest typesystem version

@majiang majiang mentioned this pull request Feb 24, 2022
Closed
@majiang
Copy link

majiang commented Feb 24, 2022

Could you merge this and release? I came here from responder too

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@waghanza waghanza waghanza approved these changes

@tomchristie tomchristie tomchristie approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aminalaee @waghanza @majiang @tomchristie