Remove `testutil` dependency by jamietanna · Pull Request #9 · oapi-codegen/iris-middleware
Navigation Menu
{{ message }}
oapi-codegen / iris-middleware Public
- Notifications You must be signed in to change notification settings
- Fork 1
Merged
Conversation
Copy link
Member
jamietanna
commented
Sep 11, 2023
jamietanna
commented
As it leads to a circular dependency when we want to consume this
middleware module from oapi-codegen itself, and as it's not doing too
much heavy lifting we can remove it.
As it leads to a circular dependency when we want to consume this middleware module from `oapi-codegen` itself, and as it's not doing too much heavy lifting we can remove it.
jamietanna
force-pushed
the
chore/testutil
branch
from
e92b9fc to
f0d2b07
Compare
jamietanna
merged commit
260c88b
into
main
jamietanna
deleted the
chore/testutil
branch
Copy link
Member Author
jamietanna
commented
Sep 11, 2023
jamietanna commented
Sep 11, 2023I've reverted this to keep the latest tag as working. Will address the fixes tomorrow.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment