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

Make FrameworkField ctor public. Fixes #1668 #1669

Merged

Conversation

pholser
Copy link
Contributor

@pholser pholser commented Jul 15, 2020

Prior to this change, custom runners could make FrameworkMethod
instances, but not FrameworkField instances. This small change
allows for both now, because FrameworkFields constructor has been
promoted to public from package-private.

@kcooney kcooney changed the title For #1668, make FrameworkField ctor public Make FrameworkField ctor public. Fixes #1668 Jul 18, 2020
@@ -9,7 +9,7 @@ private Version() {
}

public static String id() {
return "4.13-SNAPSHOT";
return "4.14-SNAPSHOT";
Copy link
Member

Choose a reason for hiding this comment

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

Do you mind moving this change it's own commit (before the other changes)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure thing! Done.

@pholser pholser force-pushed the issues/1668/make-framework-field-ctor-public branch from ff9ca34 to c1203e0 Compare July 20, 2020 14:46
Prior to this change, custom runners could make `FrameworkMethod`
instances, but not `FrameworkField` instances. This small change
allows for both now, because `FrameworkField`'s constructor has been
promoted to `public` from package-private.
@pholser pholser force-pushed the issues/1668/make-framework-field-ctor-public branch from 47f868e to 5be301c Compare July 22, 2020 13:15
@pholser
Copy link
Contributor Author

pholser commented Jul 22, 2020

Whoops. Done. Thanks again!

@kcooney kcooney merged commit 714019f into junit-team:main Aug 2, 2020
@marcphilipp marcphilipp added this to the 4.13.1 milestone Oct 11, 2020
@marcphilipp marcphilipp linked an issue Oct 11, 2020 that may be closed by this pull request
@marcphilipp marcphilipp removed this from the 4.13.1 milestone Oct 11, 2020
This was referenced Mar 15, 2021
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

Successfully merging this pull request may close these issues.

Make ctor FrameworkField(Field) public?
3 participants