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

Add .cache-ignore file support #12

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

antoniogamiz
Copy link
Collaborator

All pod-files names are first read throught 'get-pods' method, so that's
where the filter should be applied.

References: Raku/Documentable#34

All pod-files names are first read throught 'get-pods' method, so that's
where the filter should be applied.

References: Raku/Documentable#34
@JJ
Copy link
Collaborator

JJ commented Oct 8, 2019

Tests are failing, can you please check?

- Before this changes '.cache-ignore' worked the other way around (as a
  whitelist)

References:
@JJ
Copy link
Collaborator

JJ commented Oct 8, 2019

You should use IO::Paths if you want the paths to work in a OS-independent way, not the strings.

@antoniogamiz antoniogamiz force-pushed the master branch 2 times, most recently from afffb7c to c372a5a Compare October 11, 2019 16:26
@antoniogamiz
Copy link
Collaborator Author

antoniogamiz commented Oct 11, 2019

Agh, I cannot guess why that "\\" always appears.

@antoniogamiz antoniogamiz force-pushed the master branch 3 times, most recently from 6866ce1 to 824297a Compare October 11, 2019 16:50
@JJ
Copy link
Collaborator

JJ commented Feb 26, 2020

Ping?

@antoniogamiz
Copy link
Collaborator Author

Weird error in Windows. I will fix this when I have time to install a virtual machine with windows.

@antoniogamiz
Copy link
Collaborator Author

Update: rakudo/rakudo#3551 (comment)

@JJ
Copy link
Collaborator

JJ commented May 6, 2020

Still failing. Can you please check?

@JJ
Copy link
Collaborator

JJ commented Jul 29, 2020

It's failing probably due to the same reason @coke says in #22. Let's try and see.

@antoniogamiz
Copy link
Collaborator Author

This module should be deleted. Pod::From::Cache is its replacement.

@antoniogamiz
Copy link
Collaborator Author

antoniogamiz commented Jul 29, 2020

A new version of Documentable is going to be released tonight (Raku/Documentable#125). Please let me know if it fails in Mac (it should not, the cache module has been changed).

Maybe I should add a Mac to CI workflow.

@JJ
Copy link
Collaborator

JJ commented Jul 29, 2020 via email

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