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

Added composer.json file and fixed file structure #55

Merged
merged 4 commits into from Jan 2, 2019

Conversation

Projects
None yet
2 participants
@bopoda
Copy link
Owner

bopoda commented Dec 31, 2018

  • add composer.json file (needs for packagist.org)
  • move all code to src/RobotsTxtParser folder
  • move tests to tests/RobotsTxtParser folder
  • fix code style to PSR compatible format
  • improve RobotsTxtValidator (getRelativeUrl function)

ref #53

@lyrixx

lyrixx approved these changes Jan 2, 2019

Copy link
Contributor

lyrixx left a comment

👍 This is super cool. Packagist, PSR style ❤️ Thanks

.travis.yml Outdated
- 7.1
- 7.2
- 7.3
- hhvm

This comment has been minimized.

@lyrixx

lyrixx Jan 2, 2019

Contributor

usually, we drop support for hhvm. More over you have allowed failures on HHVM. So tests are running for nothing except consuming some resources, and that, that kill polar bear :D

This comment has been minimized.

@bopoda

bopoda Jan 2, 2019

Owner

Let's save polar bears 👍
hhvm is removed

@bopoda bopoda merged commit e5f59ff into master Jan 2, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@bopoda bopoda deleted the composer.json branch Jan 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment