Skip to content

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented Jul 8, 2016

Since we can't run them anyway we better exclude them properly

I'm still thinking if we are better of doing this with groups? Or different suites.
Same approach I want to take with testing the different memcache backends. Isolate them in suites. And just run them with the required backend available.

CC: @MorrisJobke

@rullzer rullzer added the 2. developing Work in progress label Jul 8, 2016
@mention-bot
Copy link

@rullzer, thanks for your PR! By analyzing the annotation information on this pull request, we identified @MorrisJobke, @DeepDiver1975 and @schiessle to be potential reviewers

* Also allow the testsuite selection in autotest.sh
@rullzer rullzer force-pushed the ci-exclude-swift branch from 80aaf78 to 1a273f8 Compare July 8, 2016 17:36
.htaccess Outdated
#### DO NOT CHANGE ANYTHING ABOVE THIS LINE ####

ErrorDocument 403 /core/templates/403.php
ErrorDocument 404 /core/templates/404.php
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👎

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fair enough..

@rullzer rullzer force-pushed the ci-exclude-swift branch from 1a273f8 to 0b5faa1 Compare July 9, 2016 18:14
@icewind1991
Copy link
Member

icewind1991 commented Jul 10, 2016

Using groups would be nicer (no need to edit phpunit.xml) but I'm fine with this if groups can't work

@rullzer
Copy link
Member Author

rullzer commented Jul 10, 2016

@icewind1991 yeah I tend to agree after thinking more about it. Let me take a look tomorrow.

@rullzer rullzer closed this Jul 13, 2016
@rullzer rullzer deleted the ci-exclude-swift branch July 13, 2016 12:21
R0Wi pushed a commit to R0Wi/server that referenced this pull request Nov 22, 2025
We do not disable caches in other places as in CLI only `/ex_apps` cache
is used.

Linked issue: nextcloud/app_api#291

Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Co-authored-by: Andrey Borysenko <andrey18106x@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants