Skip to content

Comments

Added podcast-stingers.yaml config and updated README#42

Merged
marceloams merged 3 commits intoUselessCo:developfrom
sharanyaa23:add-podcast-singers-data
Oct 21, 2025
Merged

Added podcast-stingers.yaml config and updated README#42
marceloams merged 3 commits intoUselessCo:developfrom
sharanyaa23:add-podcast-singers-data

Conversation

@sharanyaa23
Copy link
Contributor

Pull Request

Type of Change

  • Bug fix
  • New feature
  • Sound contribution
  • Configuration contribution
  • Documentation update
  • Other (specify)

Description

Added a new configuration file podcast-stingers.yaml with 5 keybinds for short podcast audio cues (intro, transition, outro, etc.). Also updated the README in the configs directory to document this new file.

Related Issue

Closes #(not applicable)

Changes Made

  • Added podcast-stingers.yaml to src/configs/
  • Updated README with a new entry for the config
  • Followed existing config format and guidelines

For Sound Contributions

  • Not applicable (using existing sound paths)

For Code Changes

  • Not applicable (no code changes)

Screenshots/Audio

Not applicable

Checklist

  • My code/contribution follows the project guidelines
  • I have tested my changes
  • I have updated documentation as needed
  • My changes don't break existing functionality

@marceloams
Copy link
Contributor

marceloams commented Oct 19, 2025

Hey @sharanyaa23! Just one thing, we don't need the configs/README.md updated. It's meant to be a general informative file, and to avoid PR conflicts, we keep it unmodified. Could you please revert those changes and keep just your config? Thanks!

@marceloams
Copy link
Contributor

Hey @sharanyaa23! Thanks for formatting the code! Before I merge, please add the sound files used in your config (you can do it in this PR or create a separate one). Thanks!

@marceloams marceloams merged commit 05c42bf into UselessCo:develop Oct 21, 2025
11 checks passed
@marceloams
Copy link
Contributor

@sharanyaa23 Thanks for your contribution! 🎉

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