-
Notifications
You must be signed in to change notification settings - Fork 13
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
[RTM] Makes the execute post actions compatible for the dcg #92
Conversation
235ea7c
to
4c34402
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
works fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Travis build is failing. After the test build works fine, i switch the label wip => rtm. |
37958b2
to
21cc2df
Compare
@discordier I have add the two dcg packages to require-dev section. So after adds the package, the travis build crashes. I have no idea, what I can do. Can you please take a look at this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please recheck .travis.yml
.
@@ -28,20 +28,23 @@ matrix: | |||
allow_failures: | |||
- env: CONTAO_VERSION=~4.10.0 | |||
|
|||
before_script: | |||
- echo "memory_limit=-1" >> ~/.phpenv/versions/$(phpenv version-name)/etc/conf.d/travis.ini | |||
sudo: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed anymore. See https://docs.travis-ci.com/user/reference/overview/#deprecated-virtualization-environments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not working. I not remove this line.
https://travis-ci.org/github/menatwork/contao-multicolumnwizard-bundle/jobs/742631080
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean by not working? Sudo is not needed anymore... I removed it from many .travis.yml
files already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh sorry. I think do you mean the before script setting
.travis.yml
Outdated
- travis_retry composer self-update && composer --version | ||
- travis_retry composer require contao/core-bundle $CONTAO_VERSION --no-update | ||
- > | ||
if [ "x${TRAVIS_TAG}" != "x" ]; then | ||
export COMPOSER_ROOT_VERSION=${TRAVIS_TAG} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert this, exporting and echoing below helps pin point problems as this PR is currently having.
We now have no clue at all what composer thinks it has installed as root package and therefore do not know how to make it install dcg.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
composer.json
Outdated
@@ -55,12 +55,14 @@ | |||
}, | |||
"require": { | |||
"php": "^5.6 || ^7.0", | |||
"contao/core-bundle": "^4.4.8", | |||
"contao/core-bundle": "~4.4.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the downgrade to 4.4.0
? AFAIR we upgraded to at least 4.4.8
for a reason (some bug that was incompatible with this extension).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
99063d5
to
1497160
Compare
…le tree widget is working now with the dcg.
1497160
to
cd33adc
Compare
@stefanheimes I temporarily removed the two necessary development packages from DC General, so we called by phone. These will be added again later. |
This is the fix for can use e.g. file tree widget with the dcg.