Closed amindadgar closed 2 months ago
The recent update changes the query parameter key from "platform"
to "name"
in the fetch_discord_platforms.py
script and its corresponding integration tests. This adjustment ensures uniformity and potentially aligns naming conventions across the codebase.
File Path | Change Summary |
---|---|
dags/analyzer_helper/discord/fetch_discord_platforms.py |
Replaced query parameter key "platform" with "name" in the fetch_all method. |
dags/analyzer_helper/tests/integration/test_integration_fetch_discord_platforms.py |
Updated test dictionaries to use the key "name" instead of "platform" for different platforms. |
In the world of code, we change and play,
A platform's name, we swap today.
Fromplatform
toname
, it's just a tweak,
For clarity's sake, a change so sleek.
Test cases follow, in line they stay,
As code evolves, we show the way.
🐰✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit