feat: expose parser options as a seperate type

[iamdevelopergirl]

Description

• Exposing the options argument in the parser function as a separate type, so that it can be reused.

Related issue(s)
Fixes #345

[github-actions]

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.

Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

[magicmatatjahu]

@iamdevelopergirl Hi! Thanks for contribution! Please also change link to the type in the functions:

parser-js/lib/parser.js

Line 129 in 07b739b

function parseFromUrl(url, fetchOptions, options) {

parser-js/lib/parser.js

Line 253 in 07b739b

async function customChannelsOperations(parsedJSON, asyncapiYAMLorJSON, initialFormat, options) {

parser-js/lib/parser.js

Line 284 in 07b739b

async function customComponentsMsgOperations(parsedJSON, asyncapiYAMLorJSON, initialFormat, options) {

I mean last options argument should point to the ParserOptions type.

Please also change the prefix of PR to the feat: :)

[iamdevelopergirl]

Addressed the review comments.

[magicmatatjahu]

@iamdevelopergirl Hello! Sorry for late review, could you add my suggestions? :)

[iamdevelopergirl]

Added the suggestions to change the type of options

[iamdevelopergirl]

Thanks for making the changes. Guess I overlooked the type of options param. :)

[magicmatatjahu]

@iamdevelopergirl Sorry, I don't see them. Did you push them or do you have only in the local repo? 😅

[iamdevelopergirl]

@magicmatatjahu Right, you got me :)
Added it now.

[magicmatatjahu]

@iamdevelopergirl No worries :) If I could, last thing. Could you rebase your repo with base (master) branch? I don't have permission to do that by myself (and it blocks me to merge), so you must do it in your side.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[iamdevelopergirl]

Yep. Done 👍

[magicmatatjahu]

@iamdevelopergirl Thanks for contribution and patience 😅

[magicmatatjahu]

@all-contributors please add @iamdevelopergirl for code

[allcontributors]

@magicmatatjahu

I've put up a pull request to add @iamdevelopergirl! 🎉

[asyncapi-bot]

🎉 This PR is included in version 1.8.0 🎉

The release is available on:

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

Your semantic-release bot 📦🚀

[iamdevelopergirl]

Cool. Thanks