Skip to content

Extract ex_moq - #7

Merged
kidq330 merged 28 commits into
masterfrom
kidq330/extract_ex_moq
Sep 8, 2026
Merged

kidq330 merged 28 commits into
masterfrom
kidq330/extract_ex_moq

Conversation

@kidq330

@kidq330 kidq330 commented Aug 26, 2026

Copy link
Copy Markdown
Member

No description provided.

@kidq330 kidq330 self-assigned this Aug 26, 2026
@kidq330 kidq330 added this to Smackore Aug 26, 2026
@kidq330 kidq330 moved this to In Review in Smackore Aug 26, 2026
@kidq330
kidq330 marked this pull request as ready for review August 26, 2026 15:33
@kidq330
kidq330 force-pushed the kidq330/extract_ex_moq branch from fa962f8 to 375add5 Compare August 26, 2026 15:35

@varsill varsill left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[NIT] Remember to remove Rust's target/ from .gitignore ;)

Comment thread mix.exs Outdated
{:membrane_aac_format, "~> 0.8.0"},
{:membrane_opus_format, "~> 0.3.0"},
# TODO: change to released version once it's public
{:ex_moq, github: "membraneframework/ex_moq", branch: "kidq330/group_loss_fixed"},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hmm shouldn't it point now to kidq330/initial?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yup that's a leftover from debugging CI

Comment thread lib/source.ex
Comment thread mix.exs
{:ex_moq, github: "membraneframework/ex_moq", branch: "kidq330/group_loss_fixed"},
{:muontrap, ">= 0.0.0", only: :test},
{:membrane_aac_plugin, "~> 0.19", only: :test},
{:muontrap, "~> 1.8", only: :test},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[NIT] I would leave ~> 1.8

Comment thread lib/source/catalog.ex Outdated
Comment thread .github/workflows/ci.yml
@kidq330
kidq330 requested a review from varsill September 3, 2026 19:02

@varsill varsill left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🥇

@kidq330
kidq330 merged commit dfab144 into master Sep 8, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Smackore Sep 8, 2026
@kidq330
kidq330 deleted the kidq330/extract_ex_moq branch September 15, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants