Skip to content

Conversation

@thdaraujo
Copy link
Contributor

@thdaraujo thdaraujo commented Nov 16, 2025

(fixes #3139)

Moves Faker::BossaNova generator under Faker::Music::BossaNova

Motivation / Background

Moving this generator under music will make it more discoverable. We've been reorganizing generators to make things more tidy.

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one change. Changes that are unrelated should be opened in separate PRs.
  • Commit message has a detailed description of what changed and why. If this PR fixes a related issue include it in the commit message. Ex: [Fix #issue-number]
  • Tests are added or updated if you fix a bug, refactor something, or add a feature.
  • Tests and Rubocop are passing before submitting your proposed changes.

@thdaraujo thdaraujo self-assigned this Nov 16, 2025
@thdaraujo thdaraujo force-pushed the ta/move-bossa-nova-generator branch from 22c96f2 to 851b5dc Compare November 16, 2025 01:17
@thdaraujo thdaraujo changed the title refactor: move Faker::BossaNova to Faker::Music::BossaNova refactor: move Faker::BossaNova under Faker::Music::BossaNova Nov 16, 2025
@thdaraujo thdaraujo force-pushed the ta/move-bossa-nova-generator branch from 851b5dc to ceea16e Compare November 16, 2025 01:22
(fixes #3139)

Moves +Faker::BossaNova+ generator under +Faker::Music::BossaNova+
- [Faker::Music::Rush](doc/music/rush.md)
- [Faker::Music::SmashingPumpkins](doc/music/smashing_pumpkins.md)
- [Faker::Music::UmphreysMcgee](doc/music/umphreys_mcgee.md)
- [Faker::Music::BossaNova](doc/music/bossa_nova.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

🎼

@thdaraujo thdaraujo merged commit efb7503 into main Nov 16, 2025
8 checks passed
@thdaraujo thdaraujo deleted the ta/move-bossa-nova-generator branch November 16, 2025 01:27
stefannibrasil pushed a commit that referenced this pull request Jan 8, 2026
)

(fixes #3139)

Moves +Faker::BossaNova+ generator under +Faker::Music::BossaNova+
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move Faker::BossaNova to Faker::Music::BossaNova

3 participants