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

Setting multiple ignore entries disables ignore feature entirely #133

Open
yuhr opened this issue Sep 22, 2021 · 0 comments
Open

Setting multiple ignore entries disables ignore feature entirely #133

yuhr opened this issue Sep 22, 2021 · 0 comments
Assignees
Labels
bug report Something isn't working

Comments

@yuhr
Copy link

yuhr commented Sep 22, 2021

🐛 Bug Report

ignore field doesn't work if there are two (or more?) entries.

To Reproduce

  1. In egg.yml, set files and ignore fields like this:

    files:
      - ./**/*.ts
    ignore:
      - ./node_modules/**/*
      - ./tests/**/*
    
  2. Run eggs publish --dry-run

Expected behavior

It excludes all files under ./node_modules and ./tests directory from the published bundle.

Actual Behavior

It lists up the files under ./node_modules and ./tests.

Environment

  • Eggs version: 0.3.8

  • Deno version: 1.14.0 (release, aarch64-apple-darwin)

  • Operating system and version:

    macOS 11.5.2
    CPU: (8) arm64 Apple M1
    

Additional context

Note that setting a single entry to ignore works expectedly.

@maximousblk maximousblk added the bug report Something isn't working label Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants