Skip to content
This repository has been archived by the owner on May 16, 2024. It is now read-only.

Update font-fira-mono-nerd-font from 2.1.0 to 2.2.1 #6649

Closed
wants to merge 3 commits into from
Closed

Update font-fira-mono-nerd-font from 2.1.0 to 2.2.1 #6649

wants to merge 3 commits into from

Conversation

b-reich
Copy link
Contributor

@b-reich b-reich commented Nov 24, 2022

Created with brew bump-cask-pr.

@BrewTestBot BrewTestBot added appcast migration needed Cask is still using `appcast` and should be migrated to using `livecheck`. bump-cask-pr PR was created using `brew bump-cask-pr` missing description Cask is missing a description, please add one. labels Nov 24, 2022
@BrewTestBot BrewTestBot removed the appcast migration needed Cask is still using `appcast` and should be migrated to using `livecheck`. label Nov 24, 2022
reitermarkus
reitermarkus previously approved these changes Nov 24, 2022
View reviewed changes
@BrewTestBot BrewTestBot removed the missing description Cask is missing a description, please add one. label Nov 24, 2022
@BrewTestBot
Copy link
Member

:shipit: @miccal has triggered a merge.

@Finii
Copy link
Contributor

Finii commented Nov 25, 2022

@b-reich Is there any reason this updates to 2.2.1 and not 2.2.2 ?

Livecheck ignored?

image

@Finii
Copy link
Contributor

Finii commented Nov 25, 2022

This package is broken (which is unrelated to this PR), btw, see ryanoasis/nerd-fonts#1003

Nerd Fonts has a cask-updater-script, maybe I should repair and activate it? Do you have an opinion on that, @b-reich ?

@Finii
Copy link
Contributor

Finii commented Nov 25, 2022

Something is strange with the CI setup @miccal.

Why does the bot merge this PR, albeit the tests failed?

image

@b-reich
Copy link
Contributor Author

b-reich commented Nov 25, 2022

This package is broken (which is unrelated to this PR), btw, see ryanoasis/nerd-fonts#1003

Nerd Fonts has a cask-updater-script, maybe I should repair and activate it? Do you have an opinion on that, @b-reich ?

You are right..
Was a mistake

@Finii love to see this working again. Updating manualy all the fonts is a bit annoying. I wrote a litte helper script. But this is not a proper solution.

@Finii
Copy link
Contributor

Finii commented Nov 25, 2022

Roger, will try to set some automatic PR creation up over at Nerd Fonts' CI.
Will ping you as CI expert (read your profile page) afterwards *frech grins*

This was referenced Nov 26, 2022
Closed
Merged
@Finii Finii mentioned this pull request Dec 14, 2022
Merged
9 tasks
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@reitermarkus reitermarkus reitermarkus left review comments

Assignees
No one assigned
Labels
bump-cask-pr PR was created using `brew bump-cask-pr`
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@b-reich @BrewTestBot @Finii @reitermarkus @miccal