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

Test the RUNTIME_IGNOREDOTFILES environment variable #78

Merged
merged 16 commits into from
Oct 30, 2019

Conversation

mdimjasevic
Copy link
Contributor

This PR tests the RUNTIME_IGNOREDOTFILES environment variable per issue #26.

@mdimjasevic mdimjasevic requested a review from neongreen October 28, 2019 10:20
@mdimjasevic mdimjasevic self-assigned this Oct 28, 2019
@mdimjasevic mdimjasevic requested a review from neongreen October 28, 2019 15:39
@neongreen
Copy link
Contributor

This includes #77, right? I think in this case you could just add the test to #77 directly.

test/Fencer/Rules/Test.hs Outdated Show resolved Hide resolved
@mdimjasevic
Copy link
Contributor Author

This includes #77, right? I think in this case you could just add the test to #77 directly.

Yes, it pulls from the same branch that PR #77 is based on, but I kept the two PRs in separate branches. They started out separately, but while switching back and forth I realized I need stuff from #77 in here as well. To avoid merging back to the original branch, I'll just keep it the way it is and merge it to master as soon as CI passes.

README.md Outdated Show resolved Hide resolved
@mdimjasevic mdimjasevic merged commit 8e523eb into master Oct 30, 2019
@mdimjasevic mdimjasevic deleted the mdimjasevic/26-runtime-ignoredotfiles-env-var branch October 30, 2019 13:06
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