swagger-js icon indicating copy to clipboard operation
swagger-js copied to clipboard

feat: Adding support to pass down resolved baseURL, to avoid repeated computation . Issue: #3537

Open sauravazad opened this issue 1 year ago • 1 comments

https://github.com/swagger-api/swagger-js/issues/3537

Description

Add support for baseURL in options for performance gains .

Motivation and Context

Issue: #3537

How Has This Been Tested?

Added supporting test case for the scenario.

Screenshots (if appropriate):

Types of changes

  • [ ] No code changes (changes to documentation, CI, metadata, etc)
  • [ ] Dependency changes (any modification to dependencies in package.json)
  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [x] My code follows the code style of this project.
  • [x] My change requires a change to the documentation.
  • [x] I have updated the documentation accordingly.
  • [x] I have added tests to cover my changes.
  • [x] All new and existing tests passed.

sauravazad avatar May 27 '24 16:05 sauravazad

I'll provide code review ASAP.

char0n avatar Oct 18 '24 12:10 char0n

:tada: This PR is included in version 3.31.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

swagger-bot avatar Nov 07 '24 11:11 swagger-bot