Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@rwaldron
Copy link
Contributor

This revision fixes some annoyances and garbage that I overlooked in the first implementation:

  1. This revised warning is never shown for --full, because that makes no sense, all files will be pushed every time.
  2. Takes into consideration that asset files and directories might indeed be directly required by the program.
  3. Doesn't display such an awful message, no more file list, etc.
  4. Puts the information in the --help text (both run and push)

Signed-off-by: Rick Waldron [email protected]

@rwaldron
Copy link
Contributor Author

cc @johnnyman727 @HipsterBrown

@rwaldron rwaldron force-pushed the ignored-assets-take-two branch 2 times, most recently from 7db95e7 to 96d7be8 Compare April 10, 2016 22:41
@johnnyman727
Copy link
Contributor

r+

@HipsterBrown
Copy link
Contributor

Very smart system. Nice that we can scan the dependency graph like that. Great update. 👍

@rwaldron rwaldron force-pushed the ignored-assets-take-two branch from 96d7be8 to 31d93a7 Compare April 11, 2016 16:24
@rwaldron
Copy link
Contributor Author

Thanks everyone

@rwaldron rwaldron merged commit 8d9d3ee into tessel:master Apr 11, 2016
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.

3 participants