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

Create .gitattributes #130

Merged
merged 1 commit into from
Dec 3, 2019
Merged

Conversation

reedy
Copy link
Contributor

@reedy reedy commented Dec 2, 2019

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 92336ee on reedy:reedy-patch-1 into 27a93dc on WhichBrowser:master.

@NielsLeenheer NielsLeenheer merged commit 5ca4f0a into WhichBrowser:master Dec 3, 2019
@reedy
Copy link
Contributor Author

reedy commented Dec 4, 2019

Thanks!

Just noticed a small potential problem...

https://github.com/WhichBrowser/Parser-PHP/blob/27f37af/composer.json#L23

Tests are autoloaded into the same namespace, so with the code as is, composer is presumably going to try and load the test files too whatever...

Needs a bit of testing (before the next release)...

If it does break things, mind if I add a PR to move tests to `WhichBrowser\Tests`` or something and update composer.json accordingly?

Or maybe try https://getcomposer.org/doc/04-schema.md#exclude-files-from-classmaps

@reedy
Copy link
Contributor Author

reedy commented Dec 5, 2019

Looks like it works fine as is, so no further changes needed

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.

None yet

3 participants