Skip to content

[Privatization] Privatize methods that are used only locally#3084

Merged
TomasVotruba merged 2 commits intomasterfrom
privatize-methods
Mar 28, 2020
Merged

[Privatization] Privatize methods that are used only locally#3084
TomasVotruba merged 2 commits intomasterfrom
privatize-methods

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Mar 28, 2020

Ref #151

class SomeClass
{
    /**
     * @api
     */
    public function run()
    {
        return $this->useMe();
    }

-   public function useMe()
+   private function useMe()
    {
    }
}

@TomasVotruba TomasVotruba force-pushed the privatize-methods branch 2 times, most recently from 0134d37 to 036f6d7 Compare March 28, 2020 15:59
@TomasVotruba TomasVotruba changed the title [WIP] [Privatization] Privatize methods that are used only locally [Privatization] Privatize methods that are used only locally Mar 28, 2020
@TomasVotruba TomasVotruba force-pushed the privatize-methods branch 11 times, most recently from d8e4c53 to 46b1758 Compare March 28, 2020 22:58
@TomasVotruba TomasVotruba merged commit e9b6a8f into master Mar 28, 2020
@TomasVotruba TomasVotruba deleted the privatize-methods branch March 28, 2020 23:11
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.

1 participant