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

HTTPS on URL link #5

Merged
merged 1 commit into from
Sep 28, 2016
Merged

HTTPS on URL link #5

merged 1 commit into from
Sep 28, 2016

Conversation

x89
Copy link
Contributor

@x89 x89 commented Sep 28, 2016

No description provided.

@tremby
Copy link
Owner

tremby commented Sep 28, 2016

See #4

@@ -73,7 +73,7 @@ while [ $# -gt 0 ]; do
url="${url%%</link>*}"
delete_hash="${response##*<deletehash>}"
delete_hash="${delete_hash%%</deletehash>*}"
echo $url
echo $url | sed 's/^http/https/'
Copy link
Owner

Choose a reason for hiding this comment

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

If you just add a colon to the end of this I'll merge it; s/^http:/https:/. I know it likely doesn't ever make any practical difference but I like to be sure.

@tremby
Copy link
Owner

tremby commented Sep 28, 2016

Great; squash the commits into one, please.

@tremby
Copy link
Owner

tremby commented Sep 28, 2016

In case you don't know how, git rebase -i HEAD\~2, then edit the line with the "Better sed regex" to start with an f for fixup (since you just want to keep the other commit message), then save. Then git push -f

@x89
Copy link
Contributor Author

x89 commented Sep 28, 2016

That should be it, yeah I'm familiar with commit squashing, just don't tend to take a strong stance on it myself!

Let me know if you'd like anything else, and thanks for the PR, I appreciate it 😘

@tremby tremby merged commit e02730a into tremby:master Sep 28, 2016
@x89
Copy link
Contributor Author

x89 commented Sep 28, 2016

Thanks! Hopefully upstream return https as well eventually.

@tremby
Copy link
Owner

tremby commented Sep 28, 2016

The code as you've written it won't mind either way, which is good.

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