Skip to content

feat: ani-sync -s arg - #694

Merged
port19x merged 8 commits into
masterfrom
ani-sync
Apr 29, 2022
Merged

feat: ani-sync -s arg#694
port19x merged 8 commits into
masterfrom
ani-sync

Conversation

@justchokingaround

Copy link
Copy Markdown
Collaborator

@port19x

port19x commented Apr 28, 2022

Copy link
Copy Markdown
Collaborator

I'll give it some thorough testing with a friend

@port19x

port19x commented Apr 28, 2022

Copy link
Copy Markdown
Collaborator

Seems to work, no issues so far

Comment thread ani-cli Outdated

@Derisis13 Derisis13 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Now it passes

@port19x

port19x commented Apr 29, 2022

Copy link
Copy Markdown
Collaborator

I'll test it again this evening and merge it if autoplay works for me

@port19x port19x linked an issue Apr 29, 2022 that may be closed by this pull request

@port19x port19x left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Confirmed working

@port19x
port19x merged commit 935a7e4 into master Apr 29, 2022
@port19x
port19x deleted the ani-sync branch April 29, 2022 18:41
billmal071 pushed a commit to billmal071/ani-cli that referenced this pull request Jun 26, 2026
* feat: ani-sync with -s argument

* chore: shellcheck, removed useless lines, added linux player_fn

* chore: shellcheck with uname

* chore: fix choke

* fix: ah yes quotes

* refactor: onliners for OS cases

* fix: auto-play

* fix: auto-play

Co-authored-by: chokerman <ivan@chokerman.local>
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.

Can we mainline ani-sync?

3 participants