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

Remove node_modules cache from .travis.yml #82

Merged
merged 1 commit into from Nov 29, 2016

Conversation

jdonaghue
Copy link
Member

@jdonaghue jdonaghue commented Nov 28, 2016

Type: bug

Description:

Remove node_modules cache from .travis.yml

Related Issue: dojo/dojo-package-template#75

Please review this checklist before submitting your PR:

  • There is a related issue
  • All contributors have signed a CLA
  • All code matches the style guide
  • The code passes the CI tests
  • Unit or Functional tests are included in the PR
  • The PR increases or maintains the overall unit test coverage percentage
  • The code is ready to be merged

@codecov-io
Copy link

Current coverage is 99.09% (diff: 100%)

Merging #82 into master will not change coverage

@@             master        #82   diff @@
==========================================
  Files            10         10          
  Lines           222        222          
  Methods           0          0          
  Messages          0          0          
  Branches         19         19          
==========================================
  Hits            220        220          
  Misses            0          0          
  Partials          2          2          

Powered by Codecov. Last update 9410145...e5407ad

@agubler agubler merged commit 924130a into dojo:master Nov 29, 2016
@dylans dylans added this to the 2016.11 milestone Nov 29, 2016
rishson added a commit to rishson/cli that referenced this pull request Nov 29, 2016
* upstream/master:
  Remove node_modules cache from .travis.yml (dojo#82)
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.

None yet

4 participants