Skip to content

Conversation

@absidue
Copy link
Member

@absidue absidue commented Sep 10, 2022


Remove unneeded electron-builder-squirrel-windows dependency

Pull Request Type

  • Other - dependency cleanup

Related issue
Related to #2113

Description
Squirrel builds were removed in #2113, this pull request removes the now unneeded electron-builder-squirrel-windows dependency.

Testing (for code that is not small enough to be easily understandable)
I tested yarn build and nothing broke.

Desktop (please complete the following information):

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 0.17.1

@absidue absidue added the PR: waiting for review For PRs that are complete, tested, and ready for review label Sep 10, 2022
@PrestonN PrestonN enabled auto-merge (squash) September 10, 2022 12:15
@PrestonN PrestonN merged commit 2d2c1e7 into FreeTubeApp:development Sep 11, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Sep 11, 2022
@absidue absidue deleted the remove-squirrel-dependency branch September 11, 2022 18:34
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Sep 13, 2022
* development:
  Create super-linter.yml (FreeTubeApp#2562)
  Update css-loader and switch to webpack 5's asset modules (FreeTubeApp#2566)
  Translated using Weblate (Spanish)
  Translated using Weblate (Portuguese (Brazil))
  Remove unused ft-intersection-observer component (FreeTubeApp#2563)
  Remove unneeded electron-builder-squirrel-windows dependency (FreeTubeApp#2560)
  Update auto-label-issue GH workflow config to use multiline (FreeTubeApp#2565)
  Update sass and sass-loader to the latest versions (FreeTubeApp#2568)
  Translated using Weblate (Romanian)
  Translated using Weblate (Spanish)
  Translated using Weblate (Spanish)
  Translated using Weblate (Spanish)
  Translated using Weblate (Bulgarian)
  Use some rich format to improve README.md readability (FreeTubeApp#2456)

# Conflicts:
#	yarn.lock
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.

5 participants