8000 ignored-*-p: Match against regular expressions by shackra · Pull Request #1260 · bbatsov/projectile · GitHub
[go: up one dir, main page]

Skip to content

ignored-*-p: Match against regular expressions#1260

Merged
bbatsov merged 9 commits intobbatsov:masterfrom
shackra:ignored-p-regexp
Jul 28, 2018
Merged

ignored-*-p: Match against regular expressions#1260
bbatsov merged 9 commits intobbatsov:masterfrom
shackra:ignored-p-regexp

Conversation

@shackra
Copy link
Contributor
@shackra shackra commented Jul 27, 2018

Buffers are created by other packages that appear on projectile's buffer lists, but they always have
some variation that makes impossible including all possible name cases, thus regular expressions
come handy for this sort of cases. This commit allows *-ignored-*-p functions to work with regular
expressions.


Before submitting a PR make sure the following things have been done (and denote this
by checking the relevant checkboxes):

  • The commits are consistent with our contribution guidelines
  • You've added tests (if possible) to cover your change(s)
  • All tests are passing (make test)
  • The new code is not generating bytecode or M-x checkdoc warnings
  • You've updated the changelog (if adding/changing user-visible functionality)
  • You've updated the readme (if adding/changing user-visible functionality)

Thanks!

shackra added 9 commits July 27, 2018 12:57
Buffers are created by other packages that appear on projectile's buffer lists, but they always have
some variation that makes impossible including all possible name cases, thus regular expressions
come handy for this sort of cases. This commit allows `*-ignored-*-p` functions to work with regular
expressions.
The function was incorrectly defined, causing to fail on tests. The correct thing to do is to call
`projectile-ignored-directories` which returns a sequence.
A new one has to be created as I mess with my fork and the branch
@shackra
Copy link
Contributor Author
shackra commented Jul 27, 2018

@bbatsov I have done all the fixes as you requested from the pull request #1177 I had to close that one because, boy, I'm really bad at Git.

Tell me what you think and please don't let this one be forgotten too :P

@bbatsov bbatsov merged commit d2b808b into bbatsov:master Jul 28, 2018
@shackra shackra deleted the ignored-p-regexp branch July 29, 2018 19:21
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

0