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

Consolidate code for HTTP client #3481

Closed
glowcloud opened this issue Apr 23, 2024 · 1 comment
Closed

Consolidate code for HTTP client #3481

glowcloud opened this issue Apr 23, 2024 · 1 comment
Assignees
@glowcloud
Labels
type: enhancement version: 3.x

Comments

@glowcloud
Copy link
Contributor

We need to consolidate the code contained inside https://github.com/swagger-api/swagger-js/blob/master/src/http/index.js.

Some parts of it seem like they could be moved to an outside module but first we need to analyse it and understand how it actually works. The refactor will help us with maintaining this code and make it easier to understand in the future.
@glowcloud glowcloud self-assigned this Apr 23, 2024
glowcloud added a commit that referenced this issue Apr 23, 2024
@glowcloud
refactor(http): refactor http client
35a2f47 
Refs #3481
@glowcloud glowcloud mentioned this issue Apr 23, 2024
Merged
glowcloud added a commit that referenced this issue Apr 24, 2024
@glowcloud
refactor(http): refactor http client (#3482)
c4198da 
Refs #3481
@glowcloud
Copy link
Contributor Author

Addressed in #3482

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@glowcloud glowcloud

Labels
type: enhancement version: 3.x
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@glowcloud