Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Removing code duplication in addon logic #3832

Merged
merged 2 commits into from
Sep 12, 2018

Conversation

tariq1890
Copy link
Contributor

No description provided.

@tariq1890 tariq1890 changed the title Removing code duplication in addon logic [WIP]Removing code duplication in addon logic Sep 12, 2018
@jackfrancis
Copy link
Member

Thanks for doing this @tariq1890! ❤️

@tariq1890 tariq1890 changed the title [WIP]Removing code duplication in addon logic Removing code duplication in addon logic Sep 12, 2018
@codecov
Copy link

codecov bot commented Sep 12, 2018

Codecov Report

Merging #3832 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #3832      +/-   ##
=========================================
+ Coverage   55.69%   55.7%   +<.01%     
=========================================
  Files         109     109              
  Lines       16249   16207      -42     
=========================================
- Hits         9050    9028      -22     
+ Misses       6411    6386      -25     
- Partials      788     793       +5

@mboersma
Copy link
Member

/lgtm

@acs-bot
Copy link

acs-bot commented Sep 12, 2018

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mboersma, tariq1890

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jackfrancis jackfrancis merged commit 3c7ebbe into Azure:master Sep 12, 2018
@ghost ghost removed the in progress label Sep 12, 2018
@tariq1890 tariq1890 deleted the addon_enabled_cleanup branch September 12, 2018 16:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants