Skip to content
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

fix: replace deprecated default download server url #67

Merged
merged 7 commits into from
Jun 13, 2022
Merged

fix: replace deprecated default download server url #67

merged 7 commits into from
Jun 13, 2022

Conversation

d7oc
Copy link
Contributor

@d7oc d7oc commented Jun 13, 2022

After some internal discussion it seems the URL has to be adjusted as version 10.10 is not available in the old default URL but in the new one contained in this PR.

@xoxys
Copy link
Contributor

xoxys commented Jun 13, 2022

Can you please rebase your PR?

@d7oc
Copy link
Contributor Author

d7oc commented Jun 13, 2022

You mean to get rid of all those merge commits in between? I thought you would squash them anyway. The branch itself was up-to-date with upstream, that's why there is only one file changed.

@xoxys
Copy link
Contributor

xoxys commented Jun 13, 2022

Yeah, it's still cluttering the PR a bit and make reviewing a merged PR after months harder to understand if it contains commits that have nothing to do with the PR itself. I'm going to merge it as it is for now, next time please use a clean branch for each PR on your fork.

@xoxys xoxys changed the title Adjusted default for download_url to new server fix: replace deprecated default download server url Jun 13, 2022
@xoxys xoxys merged commit 5c45714 into owncloud-ansible:main Jun 13, 2022
ownclouders pushed a commit that referenced this pull request Jun 13, 2022
Author: Dennis Sieben <[email protected]>
Date:   Mon Jun 13 21:32:16 2022 +0200

    fix: replace deprecated default download server url (#67)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants