Closed yueguoguo closed 7 years ago
@yueguoguo, Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA. Thanks, Microsoft Pull Request Bot
Thank you. This is a great contribution.
Please can you modify your code as follows:
dplyr
or magrittr
. This introduces a very large set of dependencies, for work that can easily be done using base R.For roxygen documentation:
@title
tag. Simply write the title of the function. This should be a single sentence. Any other words should go on a new line.@inheritParams
where possibleGreat addition, I had played with the billing APIs and I did think of some functions in this area. I can imagine some pretty cool plots could be added. The one challenge I became aware of is that it currently only supports PAYG(Not EA) so the rate card should be used with caution.
@Alanwe Very good point. Thinking of putting notes somewhere. I am using Microsoft internal subscription, and currently have no way to test other offer options. Need advice. Cheers.
Thank you!