Skip to content
This repository was archived by the owner on Oct 24, 2023. It is now read-only.

remove unused azconst methods#35

Merged
tariq1890 merged 2 commits intoAzure:masterfrom
tariq1890:remove_az_const
Nov 19, 2018
Merged

remove unused azconst methods#35
tariq1890 merged 2 commits intoAzure:masterfrom
tariq1890:remove_az_const

Conversation

@tariq1890
Copy link
Contributor

We are removing unneeded methods in the azure const file. Code cleanup.

@codecov
Copy link

codecov bot commented Nov 17, 2018

Codecov Report

Merging #35 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #35   +/-   ##
=======================================
  Coverage   48.79%   48.79%           
=======================================
  Files          91       91           
  Lines       13859    13859           
=======================================
  Hits         6763     6763           
  Misses       6486     6486           
  Partials      610      610

@mboersma
Copy link
Member

/lgtm

Copy link
Contributor

@CecileRobertMichon CecileRobertMichon left a comment

Choose a reason for hiding this comment

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

/lgtm

@tariq1890
Copy link
Contributor Author

@mboersma @CecileRobertMichon Thanks for the review. I have updated the PR with a non-breaking changes : New tests added.

@acs-bot acs-bot added the lgtm label Nov 19, 2018
Copy link
Contributor

@CecileRobertMichon CecileRobertMichon left a comment

Choose a reason for hiding this comment

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

/lgtm

@mboersma
Copy link
Member

/lgtm

@acs-bot
Copy link

acs-bot commented Nov 19, 2018

[APPROVALNOTIFIER] This PR is APPROVED

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

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [CecileRobertMichon,mboersma,tariq1890]

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

@tariq1890 tariq1890 merged commit 125d95c into Azure:master Nov 19, 2018
@tariq1890 tariq1890 deleted the remove_az_const branch November 19, 2018 19:24
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