Skip to content

Tests - #7

Closed
twokul wants to merge 1 commit into
broccolijs:masterfrom
twokul:tests
Closed

Tests#7
twokul wants to merge 1 commit into
broccolijs:masterfrom
twokul:tests

Conversation

@twokul

@twokul twokul commented Jun 7, 2015

Copy link
Copy Markdown
Contributor

No description provided.

@stefanpenner

Copy link
Copy Markdown
Collaborator

awesome

@stefanpenner

Copy link
Copy Markdown
Collaborator

we should add some additional tests:

  • that ensure changing an option (like modules: true, to false) invalidates
  • that the changing of the contents of a file invalidates

We can likely merge this before, but those two above scenarios would provide extremely high value.

@twokul

twokul commented Jun 7, 2015

Copy link
Copy Markdown
Contributor Author

@stefanpenner absolutely, just the first pass

@stefanpenner

Copy link
Copy Markdown
Collaborator

sounds good, you are a collab. Feel free to merge, and move to the next :)

zackthehuman pushed a commit to zackthehuman/broccoli-persistent-filter that referenced this pull request Aug 22, 2015
@stefanpenner

Copy link
Copy Markdown
Collaborator

#8

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.

2 participants