-
Notifications
You must be signed in to change notification settings - Fork 12
common adaptor should use new expandReferences API #306
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
Comments
Adding this to the HTTP epic - although really it's a common 2.0 epic. Since we're about to bump to common 2.0, we should do this. Just like in #315, this will break compatibility with many adaptors. But semver will protect us from the worst of the damage, and the the worst-case scenario fix is to put out a patch release with an updated package json. I think it's important that common 2.0 has a nice clean code base, because it's going to set the standard across the repo. |
Common is not going to be bumped to 2.0 right now. I'm reducing the priority on this. |
@josephjclark Can you provide a bit more information on what should be investigated. It's a bit hard to understand. |
@hunterachieng this issue still need estimates |
Yes this is mess 😅 We actually need to consolidate all 3 issues into one, and scope the exact work that's needed next. If @hunterachieng is estimating can we book a call to discuss and tidy this up? |
@josephjclark yes we can discuss it. I have removed the estimate |
1h for making the estimate and new issue with actual dev estimate. |
To be done urgently
The text was updated successfully, but these errors were encountered: