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.

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

Many "Overwriting variant method part '' for media type '...' by media type '...'" Warnings #329

Open
JMS-1 opened this issue Sep 10, 2024 · 0 comments

Comments

@JMS-1
Copy link

JMS-1 commented Sep 10, 2024

After updating this morning the generator produces quite a bit of warnings of the indicated type. This relates to the change

            const part = this.variantMethodPart(type);

            if (map.has(part)) {
              this.logger.warn(`Overwriting variant method part '${part}' for media type '${map.get(part)?.mediaType}' by media type '${type.mediaType}'.`);
            }

            map.set(part, type);

in operation.ts. I'm not quite sure if it's necessary to have the empty type in the map at all or if it may be desirable to suppress this specific warning on !part or event with a new setting.

But since we have a quite large API this warnings mess with the build and make it hard to find real problems.

Could you please check on this (should be #317).

Thanks

Jochen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant