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

Bug#388 - fix negative ignore #389

Closed
wants to merge 107 commits into from
Closed

Bug#388 - fix negative ignore #389

wants to merge 107 commits into from

Conversation

deadc0de6
Copy link
Owner

This should fix #388 and improve tests for negative ignore patterns

  • instignore for install
  • impignore for import
  • cmpignore for compare
  • upignore for ignore

In particular it should fix

  • ignore are tested twice (for is_template and for the action itself)
  • at least update is not working properly on directory

see also #298

@coveralls
Copy link

coveralls commented May 6, 2023 •
edited
Loading

Coverage Status

coverage: 91.072% (+0.1%) from 90.974% when pulling 7dca57c on bug#388 into 58318fd on master.

@deadc0de6
Copy link
Owner Author

replaced by #402

@deadc0de6 deadc0de6 closed this Sep 7, 2023
@deadc0de6 deadc0de6 deleted the bug#388 branch November 8, 2023 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bug] Negative (non-ignore) patterns refuse to match anything
2 participants
@deadc0de6 @coveralls