Skip to content
This repository was archived by the owner on Jan 13, 2022. It is now read-only.

Fix: Reduce visibility of setUp() #589

Merged
merged 1 commit into from
May 21, 2016
Merged

Fix: Reduce visibility of setUp() #589

merged 1 commit into from
May 21, 2016

Conversation

@localheinz
Copy link
Contributor

@localheinz localheinz commented May 21, 2016

This PR

  • reduces the visibility of setUp() from public to protected

🙎 This is the visibility as declared on the parent; there's no need to increase it.

@ghost ghost added the CLA Signed label May 21, 2016
@SammyK SammyK added this to the 5.2.1 milestone May 21, 2016
@SammyK SammyK merged commit fa02dcb into facebookarchive:master May 21, 2016
@SammyK
Copy link
Contributor

SammyK commented May 21, 2016

Thanks! :)

@localheinz localheinz deleted the fix/visibility branch May 21, 2016 20:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

CLA Signed

Projects

None yet

Milestone

5.2.1

Development

Successfully merging this pull request may close these issues.

2 participants

@localheinz @SammyK