Skip to content

Conversation

sroze
Copy link
Contributor

@sroze sroze commented Sep 2, 2018

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? ?
Fixed tickets ø
License MIT
Doc PR ø

AppVeyor is always failing on this test. Trying with the real HTTPS url of the image to see if it's working or if it's something else.

Copy link
Member

@fabpot fabpot left a comment

Choose a reason for hiding this comment

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

for 2.8

@sroze sroze force-pushed the filesystem-remote-file-on-app-veyor branch from 1f53aaf to 886f739 Compare September 2, 2018 17:03
@sroze sroze requested review from dunglas and xabbuh as code owners September 2, 2018 17:03
@sroze sroze changed the base branch from master to 2.8 September 2, 2018 17:03
@sroze
Copy link
Contributor Author

sroze commented Sep 2, 2018

Rebased on 2.8 👍

@fabpot
Copy link
Member

fabpot commented Sep 2, 2018

Thank you @sroze.

@fabpot fabpot merged commit 886f739 into symfony:2.8 Sep 2, 2018
fabpot added a commit that referenced this pull request Sep 2, 2018
This PR was merged into the 2.8 branch.

Discussion
----------

[Filesystem] Use the real image URL for tests

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | ?
| Fixed tickets | ø
| License       | MIT
| Doc PR        | ø

AppVeyor is always failing on this test. Trying with the real HTTPS url of the image to see if it's working or if it's something else.

Commits
-------

886f739 Use the real image URL for the filesystem tests
@sroze sroze deleted the filesystem-remote-file-on-app-veyor branch September 2, 2018 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants