Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(panel): consolidate redundant validations #9631

Merged
merged 2 commits into from
May 15, 2020

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Sep 17, 2016

  • The panel has the _validateXPosition and _validateYPosition methods which do the exact same thing, except on a different object. This moves that logic to a function in order to avoid the duplicated code.
  • Removes a use of double equals in the validation function.

CC @ErinCoughlan @bradrich

@crisbeto crisbeto added the needs: review This PR is waiting on review from the team label Sep 17, 2016
Copy link
Contributor

@ErinCoughlan ErinCoughlan left a comment

Choose a reason for hiding this comment

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

Lgtm

@crisbeto crisbeto added the needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved label Sep 26, 2016
* The panel has the `_validateXPosition` and `_validateYPosition` methods which do the exact same thing, except on a different object. This moves that logic to a function in order to avoid the duplicated code.
* Removes a use of double equals in the validation function.
@crisbeto crisbeto force-pushed the panel-redundant-checks branch from 9a23e0f to 6d80a2e Compare September 26, 2016 23:26
@crisbeto crisbeto removed the needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved label Sep 26, 2016
@crisbeto
Copy link
Member Author

@ThomasBurleson this should also be ready for presubmit.

@ThomasBurleson ThomasBurleson modified the milestones: 1.1.3, 1.2.0 Jan 1, 2017
@Splaktar Splaktar added pr: lgtm This PR has been approved by the reviewer and removed needs: review This PR is waiting on review from the team labels Apr 22, 2020
@Splaktar Splaktar assigned Splaktar and unassigned ThomasBurleson Apr 22, 2020
@Splaktar Splaktar modified the milestones: Future, 1.1.23 Apr 22, 2020
@Splaktar Splaktar added P2: required Issues that must be fixed. type: bug needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved labels Apr 22, 2020
@googlebot googlebot added the cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ label May 15, 2020
@Splaktar Splaktar removed the needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved label May 15, 2020
Copy link
Contributor

@Splaktar Splaktar left a comment

Choose a reason for hiding this comment

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

LGTM

@Splaktar Splaktar added the pr: merge ready This PR is ready for a caretaker to review label May 15, 2020
@Splaktar Splaktar merged commit a334134 into angular:master May 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ P2: required Issues that must be fixed. pr: lgtm This PR has been approved by the reviewer pr: merge ready This PR is ready for a caretaker to review type: bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants