Skip to content
This repository has been archived by the owner on May 19, 2021. It is now read-only.

Add option to set top header comment banner text for stub #1

Merged
3 commits merged into from
May 20, 2013

Conversation

clue
Copy link
Contributor

@clue clue commented May 19, 2013

This patch adds an optional setBanner($text) method to the StubGenerator in order to overwrite the default banner text.

*
* @var string.
*/
private $banner = 'Generated by Box.
Copy link

Choose a reason for hiding this comment

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

I know this is a nitpick, but could you put this property in alphabetical order?

@ghost
Copy link

ghost commented May 20, 2013

I really appreciate this feature!

Once the changes have been made I will merge it.

@clue
Copy link
Contributor Author

clue commented May 20, 2013

Glad to hear you're interested 👍 Just pushed an update that implements your comments.

ghost pushed a commit that referenced this pull request May 20, 2013
Add option to set top header comment banner text for stub
@ghost ghost merged commit 69a742e into box-project:master May 20, 2013
@ghost
Copy link

ghost commented May 20, 2013

Thank you!

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant