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 usage of allowEmpty when also using include / exclude #106

Merged
merged 6 commits into from
Feb 8, 2019
Merged

Fix usage of allowEmpty when also using include / exclude #106

merged 6 commits into from
Feb 8, 2019

Commits on Jan 23, 2019

  1. Add failing test for include/exclude and empty input node.

    @chriskrycho
    chriskrycho committed Jan 23, 2019
    Configuration menu
    Copy the full SHA
    c49cf8d View commit details
    Browse the repository at this point in the history

  2. Handle empty src dir if allowEmpty with include/exclude.

    @chriskrycho
    chriskrycho committed Jan 23, 2019
    Configuration menu
    Copy the full SHA
    a76c4ee View commit details
    Browse the repository at this point in the history

  3. Add test for available input node at missing source.

    @chriskrycho
    chriskrycho committed Jan 23, 2019
    Configuration menu
    Copy the full SHA
    74f3454 View commit details
    Browse the repository at this point in the history

  4. Make an empty dir if missing but allowEmpty. 

    Avoid trolling anything downstream which may *expect* a folder to be
present, i.e. which does not understand or respect `allowEmpty` itself.
    @chriskrycho
    chriskrycho committed Jan 23, 2019
    Configuration menu
    Copy the full SHA
    67b2e2c View commit details
    Browse the repository at this point in the history

  5. Add test for creating nested output when missing input node.

    @chriskrycho
    chriskrycho committed Jan 23, 2019
    Configuration menu
    Copy the full SHA
    6f031e3 View commit details
    Browse the repository at this point in the history

  6. Fix indentation in latest-added test.

    @chriskrycho
    chriskrycho committed Jan 23, 2019
    Configuration menu
    Copy the full SHA
    a793c0b View commit details
    Browse the repository at this point in the history