Skip to content

fix: remove open block comments #94

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

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
8246b97
fix: remove open block comments
ondrejhlavacek Aug 15, 2017
51c1cc4
fix: coding style
ondrejhlavacek Aug 15, 2017
4b5076f
feat: turn on error reporting
ondrejhlavacek Feb 12, 2019
80abd6f
fix: trailing spaces
ondrejhlavacek Feb 12, 2019
79405f4
feat: failing trailing open comment test
ondrejhlavacek Feb 12, 2019
75608b1
fix: the test should not assume removing the trailing closing comment
ondrejhlavacek Feb 12, 2019
79a471b
fix: missing string index
ondrejhlavacek Feb 12, 2019
474d064
fix: remove buggy versions, add newer versions
ondrejhlavacek Feb 12, 2019
a101cec
fix: remove unsupported php versions
ondrejhlavacek Feb 12, 2019
a2d6fe7
fix: remove further unsupported php versions
ondrejhlavacek Feb 12, 2019
da5373d
fix: use tabs
ondrejhlavacek Feb 12, 2019
c33f7c1
Merge pull request #1 from keboola/najlos-trailing-comment
ondrejhlavacek Feb 13, 2019
24fab8d
format sql in removeComments method is optional choice
ondrajodas Sep 3, 2020
29bb19a
Merge pull request #2 from keboola/ondra-format-in-remove-comments
ondrajodas Sep 3, 2020
20913af
Run tests from PHP 5.6, 7.1, 7.4
michaljurecko Dec 11, 2020
ad0f880
Generalization of comment implementation
michaljurecko Dec 11, 2020
d5dad3c
Ignore composer.lock
michaljurecko Dec 11, 2020
0a1f73a
Add test for Synapse temp table syntax
michaljurecko Dec 11, 2020
94fee9b
Improve code readability
michaljurecko Dec 11, 2020
0643d07
Merge pull request #3 from keboola/webrouse-COM-570-fix-synapse-comments
michaljurecko Dec 11, 2020
f056764
Added method setBoundaries
michaljurecko Dec 11, 2020
03c7561
Merge pull request #4 from keboola/webrouse-COM-570-fix-synapse-comme…
michaljurecko Dec 11, 2020
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
.idea/
vendor/
composer.lock
11 changes: 6 additions & 5 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
language: php
php:
- 5.6
- 5.5
- 5.4
- 5.3
- hhvm
- 7.1
- 7.4

script: phpunit --coverage-text
script:
- if [[ ${TRAVIS_PHP_VERSION:0:3} != "5.6" ]]; then phpenv config-rm xdebug.ini; fi
- composer install
- ./vendor/bin/phpunit --coverage-text
6 changes: 6 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,12 @@ A lightweight php class for formatting sql statements.

It can automatically indent and add line breaks in addition to syntax highlighting.

For local development run:
```
docker run --rm -it -v $(pwd):/app --entrypoint "" prooph/composer:5.6 composer install
docker run --rm -it -v $(pwd):/app --entrypoint "" prooph/composer:5.6 ./vendor/bin/phpunit
```

History
============

Expand Down
6 changes: 6 additions & 0 deletions bootstrap.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
<?php

require __DIR__.'/vendor/autoload.php';

ini_set('display_errors', true);
ini_set('error_reporting', E_ALL);
4 changes: 2 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
"license": "MIT",
"type": "library",
"require": {
"php": ">=5.2.4"
"php": ">=5.6"
},
"require-dev": {
"phpunit/phpunit": "3.7.*"
"phpunit/phpunit": "^5.0"
},
"authors": [
{
Expand Down
Loading