Skip to content

Move mediorum into top level audius-protocol go module - #9938

Merged
endline merged 11 commits into
mainfrom
endl/pkg-mediorum
Oct 4, 2024
Merged

Move mediorum into top level audius-protocol go module#9938
endline merged 11 commits into
mainfrom
endl/pkg-mediorum

Conversation

@endline

@endlineendline commented Oct 2, 2024

Copy link
Copy Markdown
Contributor
  • mediorum is now a top level pkg and is instantiated by the cmd/mediorum target.
  • also reduces resulting docker image size by 75%
mediorum edge 83f76c98f1ad 24 hours ago 2GB
mediorum latest f2d7285061b9 22 seconds ago 542MB

TEST

# native build
go build -o bin/mediorum cmd/mediorum/main.go
# local dev
make mediorum-dev
# test dev & ci tooling
audius-compose test mediorum
audius-compose build mediorum
audius-compose build --prod mediorum
# test bpm and key analysis
docker run --rm -ti -v ./file_example_WAV_10MG.wav:/wav/example.wav --entrypoint /bin/bash audius-protocol-mediorum -c "analyze-key /wav/example.wav"
G minor
docker run --rm -ti -v ./file_example_WAV_10MG.wav:/wav/example.wav --entrypoint /bin/bash audius-protocol-mediorum -c "analyze-bpm /wav/example.wav"
BPM: 120.904
Confidence: 3.02578

@changeset-bot

changeset-botBot commented Oct 2, 2024

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 403a444

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@gitguardian

gitguardianBot commented Oct 2, 2024

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 10 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
11648675TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/run.goView secret
11648675TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/mediorum.goView secret
11648678TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/mediorum.goView secret
11648678TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/run.goView secret
11648679TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/mediorum.goView secret
11648679TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/run.goView secret
11648680TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/run.goView secret
11648680TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/mediorum.goView secret
11648681TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/mediorum.goView secret
11648681TriggeredGeneric High Entropy Secretd40d74fpkg/mediorum/run.goView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/endl/pkg-mediorum

2 similar comments
@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/endl/pkg-mediorum

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/endl/pkg-mediorum

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/endl/pkg-mediorum

@alecsavvyalecsavvy 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.

epic

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/endl/pkg-mediorum

@endline
endlineforce-pushed the endl/pkg-mediorum branch 3 times, most recently from 1178208 to 8daff90CompareOctober 3, 2024 21:14
@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/endl/pkg-mediorum

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.

is this file supposed to be here?

Comment threadMakefile Outdated
Comment threadcmd/audiusd/main.go Outdated
@endline

Copy link
Copy Markdown
ContributorAuthor

adding last makefile changes

satisfied with PASS here

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/endl/pkg-mediorum

@endline
endline merged commit c30c411 into mainOct 4, 2024
@endline
endline deleted the endl/pkg-mediorum branch October 4, 2024 15:32
audius-infra pushed a commit that referenced this pull request Oct 7, 2024
[9d45442] [C-5046, C-5047] Add comment notification settings (#9882) Dylan Jeffers
[85ff920] Batch chat blast notifications by blastId and userId (#9949) Reed
[c30c411] Move mediorum into top level audius-protocol go module (#9938) endline
schottra added a commit that referenced this pull request Oct 7, 2024
…ient-v1-feed
* origin/main: (22 commits)
Audius Protocol v0.7.6
[C-5046, C-5047] Add comment notification settings (#9882)
[PROTO-1944] Dynamically resize Qm image variants (#9960)
Fix mobile lint (#9968)
Upgrade mobile apps to 117 (#9967)
Audius Client (Web and Mobile) v1.5.102
Move core into top level audius-protocol go module (#9940)
Fix bottom sheet keyboard avoiding + compose ui (#9964)
[C-5097] Add backspace handling back to mobile ComposerInput (#9962)
[C-4994] Add user autocomplete and tagging to mobile ComposerInput (#9952)
Slim image missing ca-certs (#9961)
Batch chat blast notifications by blastId and userId (#9949)
[QA-1748] Fix misc sign up issues (#9957)
Move mediorum into top level audius-protocol go module (#9938)
core: cleanup sandboxes (#9946)
Audius Protocol v0.7.5
[PAY-3493] Fix download gating on usdc feature flag (#9945)
[C-5025] Add compose ui helper text (#9955)
[C-5082] Comments desktop replies tweaks (#9954)
[C-5068] Convert comments a-query to tanstack-query (#9944)
...
samgutentag pushed a commit that referenced this pull request Nov 1, 2024
samgutentag pushed a commit that referenced this pull request Nov 1, 2024
[9d45442] [C-5046, C-5047] Add comment notification settings (#9882) Dylan Jeffers
[85ff920] Batch chat blast notifications by blastId and userId (#9949) Reed
[c30c411] Move mediorum into top level audius-protocol go module (#9938) endline
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@endline@audius-infra@phelpsdb@alecsavvy