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

Applied EditorBrowsable to now obsoleted aggregation types #2784

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

gfoidl
Copy link
Contributor

@gfoidl gfoidl commented Nov 17, 2023

Fixes #2773

Please see description in the issue for the rationale for this change.

@gfoidl
Copy link
Contributor Author

gfoidl commented Nov 17, 2023

Spell-check fails with the same error as #2774 (comment)

ApplicationError: Unsupported NodeJS version (16.20.2); >=18 is required

Can we either bring the other PR in or have a separate PR to update the spell-check to Node 18?
In order to avoid repeating that update in every PR and introducing possible merge conflicts.

@mysticmind
Copy link
Member

@gfoidl I will have a look shortly and sort out the spell-check stuff.

@gfoidl
Copy link
Contributor Author

gfoidl commented Nov 17, 2023

The spell-check is basically just 647a25b

@mysticmind
Copy link
Member

The spell-check is basically just 647a25b

I know, have fixed the spell checker one which is failing. Please rebase your PR with master, it should be fine.

Copy link
Member

@mysticmind mysticmind left a comment

Choose a reason for hiding this comment

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

@gfoidl Looks good to me 👍

@mysticmind mysticmind added this to the 6.3.1 milestone Nov 17, 2023
@mysticmind mysticmind added the 6.x label Nov 17, 2023
@mysticmind mysticmind merged commit c745535 into JasperFx:master Nov 17, 2023
13 checks passed
@gfoidl gfoidl deleted the editor-browsable branch November 17, 2023 14:04
@oskardudycz oskardudycz modified the milestones: 6.3.1, 6.4.0 Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply EditorBrowsable to now obsolete members like SelfAggregate
3 participants