Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wip/portuguese #730

Closed
wants to merge 605 commits into from
Closed

Wip/portuguese #730

wants to merge 605 commits into from

Conversation

dougtakeuti
Copy link
Contributor

So I added a few Portuguese translations. Does that work?

jasoncalabrese and others added 30 commits July 1, 2015 00:03
added some missing sbx.scaleBg's
Notifications generated by BWP were missing a sbx.scaleBg() around raw values
fix lots of little issues reported by codacy
Fix for the AR2_USE_RAW option and 0 values
remove some duplication when formatting messages
less code in client.js and using it for messages too
converted the direction to a plugin
fix a bug that caused auto ack'd messaages to be sent over and over
@codecov-io
Copy link

Current coverage is 83.97%

Merging #730 into dev will increase coverage by +0.80% as of 28aa8a5

@@              dev    #730   diff @@
=====================================
  Files          46      46       
  Stmts        2430    2445    +15
  Branches      486     488     +2
  Methods       380     380       
=====================================
+ Hit          2021    2053    +32
+ Partial        18      17     -1
+ Missed        391     375    -16

Review entire Coverage Diff


Uncovered Suggestions

  1. +0.57% via lib/mqtt.js#128...141
  2. +0.49% via lib/utils.js#31...42
  3. +0.49% via lib/mqtt.js#292...303
  4. See 7 more...

Powered by Codecov

@@ -16,17 +16,17 @@ function storage (collection, storage, pushover) {
obj.created_at = created_at.toISOString();

var preBolusCarbs = '';
if (obj.preBolus > 0 && obj.carbs) {
if (obj.preBolus != 0 && obj.carbs) {
Copy link
Member

Choose a reason for hiding this comment

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

@jasoncalabrese
Copy link
Member

Closing this while we get a PR that targets the dev branch

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.

8 participants