feat: adding interpreter options ignoreAddtionalProperties and ignoreAddtionalItems

[artur-ciocanu]

Description
This is an enhancement that would allow anyone to override default interpreter behavior for additionalProperties, and additionalItems.

A typical example using these interpreter options would look like this:

const generator = new JavaGenerator({
  processorOptions: {
    interpreter: {
      ignoreAdditionalProperties: true,
      ignoreAdditionalItems: true  
    }
  }
});

**Related issue(s)**
https://github.com/asyncapi/modelina/issues/916

[artur-ciocanu]

@200Puls feel free to check this feature. I hope it will land soon in a stable release. For now we decide to start small and allow changing the behavior of additionalProperties and additionalItems via interpreter options.

[artur-ciocanu]

@jonaslagoni please take a look when you have a chance. Thank you! 🙇

[coveralls]

Pull Request Test Coverage Report for Build 4006812059

• 2 of 4 (50.0%) changed or added relevant lines in 2 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.05%) to 92.338%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/interpreter/InterpretAdditionalItems.ts	1	2	50.0%
src/interpreter/InterpretAdditionalProperties.ts	1	2	50.0%

Totals
Change from base Build 4000693541:	-0.05%
Covered Lines:	4727
Relevant Lines:	4988

---

💛 - Coveralls

[200Puls]

Hi @artur-ciocanu Thanks for the notification.

This matches my use case too. I want to disable both of these.

We have started using the build from the next branch for some other features.

Will this be merged into both?

[artur-ciocanu]

Hi @artur-ciocanu Thanks for the notification.

This matches my use case too. I want to disable both of these.

We have started using the build from the next branch for some other features.

Will this be merged into both?

The next branch has been merged to master. I believe @jonaslagoni plans to release Modelina v1 either today or this week.

[200Puls]

The next branch has been merged to master. I believe @jonaslagoni plans to release Modelina v1 either today or this week.

Ah cool I totally missed that.

[artur-ciocanu]

@jonaslagoni thanks for merging master. When you have a chance could please take a look. Thank you.

[jonaslagoni]

Just a few comments, otherwise it looks good 👍

[artur-ciocanu]

@jonaslagoni I have addressed your comments, let me know if it looks good.

[jonaslagoni]

Just some doc changes 🙂

[jonaslagoni]

LGTM 👍

[jonaslagoni]

/rtm

[sonarcloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
2 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[asyncapi-bot]

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

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

Your semantic-release bot 📦🚀

[artur-ciocanu]

@200Puls it is finally out, please check it out!

[200Puls]

@200Puls it is finally out, please check it out!

Hi @artur-ciocanu works for me. Thanks for your effort!