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.

Sign up for GitHub

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

Jump to bottom

Fix for #4, #8 greenkeeper updates #6

Merged
merged 23 commits into from
May 25, 2016
Merged

Fix for #4, #8 greenkeeper updates #6

merged 23 commits into from
May 25, 2016

Conversation

zuker
Copy link
Contributor

@zuker zuker commented Mar 28, 2016

No description provided.

greenkeeperio-bot and others added 10 commits March 25, 2016 22:02
@zuker zuker mentioned this pull request Mar 28, 2016
Closed
@codecov-io
Copy link

codecov-io commented Mar 28, 2016 •
edited
Loading

Current coverage is 88.77%

Merging #6 into master will increase coverage by 0.54%

  1. 2 files (not in diff) in src/api/services were modified. more
    • Hits +7
  2. File src/index.js was modified. more
    • Misses 0
    • Partials 0
    • Hits +5
@@           master         #6   diff @@
========================================
  Files           8          8          
  Lines         170        187    +17   
  Methods         0          0          
  Branches        0          0          
========================================
+ Hits          150        166    +16   
- Misses         20         21     +1   
  Partials        0          0

Powered by Codecov. Last updated by 2c50973...5a70839

@jaumard jaumard mentioned this pull request Mar 30, 2016
Closed
@zuker
Fixed #8 "<ModelName> is already defined in the registry" error on au…
66651e5 
…toreload
@zuker
Merge pull request #19 from zuker/dev
9583460 
Fixed #8 "<ModelName> is already defined in the registry" error on autoreload
@zuker zuker changed the title Fix for #4, greenkeeper updates Fix for #4, #8 greenkeeper updates Mar 31, 2016
@zuker zuker mentioned this pull request Mar 31, 2016
Closed
greenkeeperio-bot and others added 7 commits April 20, 2016 16:41
@greenkeeperio-bot @zuker
chore(package): update smokesignals to version 1.0.6 (#21)
53d8f7c 
Update smokesignals to version 1.0.6 🚀
@zuker
chore(package): update smokesignals to version 1.0.6 (#21) (#23)
26c03c2 
Update smokesignals to version 1.0.6 🚀
@zuker
chore(package): update smokesignals to version 1.0.6 (#21) (#23) (#24)
2a82996 
Update smokesignals to version 1.0.6 🚀
@greenkeeperio-bot @zuker
chore(package): update babel-eslint to version 6.0.4 (#25)
bca37c9 
https://greenkeeper.io/
@zuker
Greenkeeper babel eslint 6.0.4 (#27)
4f05c6b 
* chore(package): update smokesignals to version 1.0.6 (#21)

Update smokesignals to version 1.0.6 🚀

* chore(package): update smokesignals to version 1.0.6 (#21) (#23) (#24)

Update smokesignals to version 1.0.6 🚀

* chore(package): update babel-eslint to version 6.0.4 (#25)

https://greenkeeper.io/
@zuker
updated dependencies
0a58b1f
@zuker
Merge branch 'dev'
b7fc993
jaumard
jaumard reviewed May 17, 2016
View reviewed changes
package.json
"joi": "^8.1.0",
"knex": "^0.11.3",
"lodash": "^4.12.0",
"trailpack": "^1.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

Normally you don't need trailpack dependency because you extends trailpack-datastore. Can you check ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for pointing that! The trailpack dependency removed.

@jaumard jaumard merged commit ecbc13b into trailsjs:master May 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@zuker @codecov-io @jaumard @greenkeeperio-bot