Skip to content

Conversation

@fasterthanlime
Copy link
Contributor

Also fixes typo :)

Can confirm that this does wonder for our desktop app

Squirrel.Windows checks the RELEASES file and happily picks the delta update that is much, much smaller than the full one.

(It still downloads it twice for some reason (presumably to get changelogs?), though, but that's okay — two times 60kb is still pretty small :shipit:)

@fasterthanlime
Copy link
Contributor Author

(Also, sorry if my initial comments sounded harsh/entitled, that wasn't my intention at all! 🌰 is a great piece of software and I'm really happy you made it ^_^)

@SamyPesse
Copy link
Member

Thanks, I'll run some tests with our application in the coming days then merge the PR 🚀

Copy link
Member

Choose a reason for hiding this comment

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

Hum?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ha, I didn't expect the PR to be based on master. Will fix later today, ignore for now

@fasterthanlime
Copy link
Contributor Author

By the way there is an issue with this PR that is fixed by itchio@650d399

The other commit is unrelated to the PR, sorry about the noise!

@fasterthanlime
Copy link
Contributor Author

Closed in favor of #24

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