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

Update bazarr 1.0.2 => 1.0.3 #5139

Closed
wants to merge 1 commit into from

Conversation

smaarn
Copy link
Contributor

@smaarn smaarn commented Feb 26, 2022

Description

Update bazarr 1.0.2 to 1.0.3

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Package update

@smaarn smaarn marked this pull request as ready for review February 27, 2022 14:21
@smaarn
Copy link
Contributor Author

smaarn commented Apr 3, 2022

Rebased at head (for followups if required)

@smaarn
Copy link
Contributor Author

smaarn commented Apr 30, 2022

One last rebase "for the road".

Quick (and open question): is there anything preventing from releasing this (tiny) PR ? I am asking because the 1.0.4 version has just been released and was planning to prepare another PR (to make sure to not compromise this one in case other issue arises) but if there's anything I need to do to ease review / merge process...

@th0ma7
Copy link
Contributor

th0ma7 commented Apr 30, 2022

I don't see any blocker with this "tiny" and trivial PR. All good with me.

@smaarn
Copy link
Contributor Author

smaarn commented May 1, 2022

Just checked and it seems that there is an issue with the PLIST.auto feature.
Will be checking.

@th0ma7
Copy link
Contributor

th0ma7 commented May 1, 2022

There is an issue that I opened on this (and completely forgot to follow-up). #5233

@smaarn
Copy link
Contributor Author

smaarn commented May 1, 2022

There is an issue that I opened on this (and completely forgot to follow-up). #5233

@th0ma7 I think I found a fix here: #5244

@smaarn smaarn force-pushed the bazarr/1.0.3-upgrade branch 2 times, most recently from 68aea47 to 70ddd94 Compare May 3, 2022 05:26
@smaarn
Copy link
Contributor Author

smaarn commented May 3, 2022

Rebased at head.

@smaarn smaarn mentioned this pull request May 8, 2022
5 tasks
@smaarn
Copy link
Contributor Author

smaarn commented May 8, 2022

Closing PR as superseded by #5259

@smaarn smaarn closed this May 8, 2022
@smaarn smaarn deleted the bazarr/1.0.3-upgrade branch May 8, 2022 17:13
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