Skip to content

Refactor duplication#792

Merged
RobbieTheWagner merged 3 commits intomasterfrom
refactor-duplication
May 1, 2018
Merged

Refactor duplication#792
RobbieTheWagner merged 3 commits intomasterfrom
refactor-duplication

Conversation

@RobbieTheWagner
Copy link
Copy Markdown
Member

Just some light refactoring to remove a bit of duplication

Comment thread ember_debug/mixins/port-mixin.js Outdated
removePortListeners() {
/**
* Setup or tear down port listeners. Call on `init` and `willDestroy`
* @param onOrOff 'on' or 'off' the functions to call i.e. port.on or port.off for adding or removing listeners
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@param {String} onOrOff

Comment thread app/controllers/component-tree.js Outdated
expandAll() {
/**
* Expand or collapse all component nodes
* @param {boolean} expanded If true, expanded, if false, collapsed
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

{Boolean}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Is that a standard to capitalize the types?

@RobbieTheWagner RobbieTheWagner merged commit 02351d3 into master May 1, 2018
@RobbieTheWagner RobbieTheWagner deleted the refactor-duplication branch May 1, 2018 12:49
teddyzeenny pushed a commit that referenced this pull request May 25, 2018
cyril-sf pushed a commit to cyril-sf/ember-inspector that referenced this pull request Mar 30, 2022
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.

2 participants