Added fix for basic relative URLs in servers

[dipsmishra]

This adds support for basic relative URLs in servers
Note: this still does not support variables/variable substitutions in URLs

[dipsmishra]

This resolves #59

[dipsmishra]

@kobik @kibertoad @tamlyn

Can one of you please review and merge if it looks good?
Thank you!

[deepakmonga2408]

Any update on this fix? This PR resolves the relative path errors and all tests pass. It would be great if you can review, merge and release.

[kobik]

Hi @dipsmishra,

Thanks for the PR!

I hope I'll have the time to look on the nezt couple of days.

[kobik]

Thanks @dipsmishra !

[kobik]

@dipsmishra can you please update CHANGELOG.md?

[dipsmishra]

@dipsmishra can you please update CHANGELOG.md?

Yes, sure. Should I update the version in package.json to 2.0.5 too? I am not entirely sure about your release process and hence, checking.

[kobik]

No need to update the json file as I use 'npm version' command for bumping and tagging before publishing it

[dipsmishra]

Thanks for the clarification, @kobik - I have updated the PR accordingly and also pulled in the latest changes from 'master' to get it ready for merge.

[deepakmonga2408]

@kobik, Can you please review and merge this PR?

[dipsmishra]

@kobik, Is there anything else needed to get this PR merged? Please let me know.

[kibertoad]

CI is currently misconfigured for the repo.
@kobik Could you please either drop travis and enable CircleCI for this repo (on their end, GitHub side is correct, I believe), or give me admin permissions for the repo so that I could do so?

[kobik]

@dipsmishra should be fixed now

[kibertoad]

@kobik

> api-schema-builder@2.0.4 coveralls /root/project
> cat ./coverage/lcov.info |./node_modules/.bin/coveralls


/root/project/node_modules/coveralls/bin/coveralls.js:19
      throw err;
      ^
Bad response: 422 {"message":"Couldn't find a repository matching this job.","error":true}
(Use `node --trace-uncaught ...` to show where the exception was thrown)

Any ideas?
Wonder if we should disable coverage reporting for now to unblock the PR.

[kobik]

fixed according to this lemurheavy/coveralls-public#632.

i don't like it too much as i was also required that it would allowed also in PR in order to work here.

[deepakmonga2408]

@kobik any update on pushing this PR through?

[kibertoad]

Thank you!

[github-actions]

🎉 This PR is included in version 2.0.6 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀