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

issue 2015 #2016

Merged
merged 1 commit into from
Oct 5, 2018
Merged

issue 2015 #2016

merged 1 commit into from
Oct 5, 2018

Conversation

imjayanthka
Copy link
Contributor

updated http to https

@tzarebczan
Copy link
Contributor

@imjayanthka thanks for the contribution! Can we send your some LBC in appreciation?

@imjayanthka
Copy link
Contributor Author

sure

@neb-b
Copy link

neb-b commented Oct 5, 2018

Awesome! @imjayanthka Could you squash these into one commit? And also add changelog entry?

Then we can merge it 🙂

@imjayanthka
Copy link
Contributor Author

sure give me a sec

@imjayanthka
Copy link
Contributor Author

git commit scripts fail on flow focus-check --color always

@neb-b
Copy link

neb-b commented Oct 5, 2018

I'm guessing it's because some of these files have Flow errors. It's fine to ignore them for this PR, if we want to fix any that would be great too!

For now you can do git commit -m "update to https" --no-verify

The --no-verify flag skips flow checks

@imjayanthka
Copy link
Contributor Author

does this work?

CHANGELOG.md Outdated
@@ -11,7 +11,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/).
### Changed
* Make tooltip smarter ([#1979](https://github.com/lbryio/lbry-desktop/pull/1979))
* Change channel pages to have 48 items instead of 10 ([#2002](https://github.com/lbryio/lbry-desktop/pull/2002))

* update to http ([#2016](https://github.com/lbryio/lbry-desktop/pull/2016))
Copy link

Choose a reason for hiding this comment

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

https*

@neb-b
Copy link

neb-b commented Oct 5, 2018

Yep. Just a small typo

@neb-b neb-b merged commit 25d4d09 into lbryio:master Oct 5, 2018
@neb-b neb-b added the hacktoberfest Welcome to Hacktoberfest label Oct 14, 2018
@tzarebczan
Copy link
Contributor

@imjayanthka please don't forget to reach out to us so we send you some LBC in appreciation for the PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest Welcome to Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants