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

A few small follow-up changes from v2.18.0 #4591

Merged
merged 4 commits into from
Sep 24, 2020
Merged

Conversation

glasser
Copy link
Member

@glasser glasser commented Sep 23, 2020

Review individual commits.

Copy link
Member

@abernix abernix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

This `extends` declaration was added during the development of v2.18.0 for
reasons that turned out to be unnecessary. The declaration makes some
valid plugin implementation no longer type-check, so I'm removing it.

Fixes #4585.
This one usage-reporting option was hidden in the middle of a bunch of
schema-reporting options so I missed it in the handling of legacy options.
This error generally should arise when people try to use the unexported AS 1.x
API instead of the 2.x API. (If we find other cases that trigger this error we
can update the linked issue.)

Fixes #4588.
@glasser glasser force-pushed the glasser/v2.18.0-followup branch from 40b4d51 to 8492f62 Compare September 24, 2020 16:19
@glasser glasser merged commit a50a4be into main Sep 24, 2020
@glasser glasser deleted the glasser/v2.18.0-followup branch September 24, 2020 16:27
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants