Skip to content

Clarify reference on inferred object-private parameter #22621

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 18, 2025
Merged

Clarify reference on inferred object-private parameter #22621

merged 1 commit into from
Feb 18, 2025

Conversation

som-snytt
Copy link
Contributor

@som-snytt som-snytt commented Feb 17, 2025 •
edited
Loading

Document the status quo.

Documents #22620

@som-snytt som-snytt marked this pull request as ready for review February 17, 2025 20:29
@Gedochao Gedochao requested a review from sjrd February 18, 2025 12:01
sjrd
sjrd approved these changes Feb 18, 2025
View reviewed changes
Copy link
Member

@sjrd sjrd left a comment

Choose a reason for hiding this comment

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

It's always better to document the status quo than not at all.

But it doesn't preclude figuring out whether the underlying issue is really as designed or just ... the status quo.

@sjrd sjrd merged commit 65ca99d into scala:main Feb 18, 2025
29 checks passed
@som-snytt som-snytt deleted the issue/22620-class-param-variance branch February 18, 2025 16:48
@tgodzik tgodzik mentioned this pull request Apr 22, 2025
Merged
@som-snytt som-snytt mentioned this pull request Jun 16, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sjrd sjrd sjrd approved these changes

Assignees

@sjrd sjrd

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@som-snytt @sjrd