#155: Disable max number of arguments Ruff Linting Rule [PLR0913]

[berkaykrc]

Description

This pull request includes several changes across multiple files to improve code readability by removing the # noqa: PLR0913 comment from function definitions and adding PL0913 to ruff.toml to ignore it.

Changes to function definitions:

• Backend/app/spotify_electron/playlist/playlist_controller.py: Modified the update_playlist function to remove the # noqa: PLR0913 comment.
• Backend/app/spotify_electron/playlist/playlist_repository.py: Modified the create_playlist function to remove the # noqa: PLR0913 comment.
• Backend/app/spotify_electron/playlist/playlist_service.py: Modified the update_playlist function to remove the # noqa: PLR0913 comment.
• Backend/app/spotify_electron/song/blob/song_repository.py: Modified the create_song function to remove the # noqa: PLR0913 comment.
• Backend/tests/test_API/api_test_artist.py: Modified the update_artist function to remove the # noqa: PLR0913 comment.
• Backend/tests/test_API/api_test_user.py: Modified the update_user function to remove the # noqa: PLR0913 comment.

Configuration updates:

• Backend/ruff.toml: Added PLR0913 to the ignore list in the ruff configuration file.

Commit type

• chore: updating grunt tasks etc; no production code change.

Issue

#155

Potential Impact

Functions with many arguments are harder to understand, maintain, and call. Consider refactoring functions with many arguments into smaller functions with fewer arguments, or using objects to group related arguments.
Consider changing these setting in ruff.toml:
https://docs.astral.sh/ruff/settings/#lint_pylint_max-args

Assigned

@AntonioMrtz

[AntonioMrtz]

Hi @berkaykrc :),

We have CONTRIBUTING and README files that list all the contributors to the project. You can add yourself to both files; this will show that you’ve contributed to the project and is our way of thanking those who help with the project.

[berkaykrc]

Hi @berkaykrc :),

We have CONTRIBUTING and README files that list all the contributors to the project. You can add yourself to both files; this will show that you’ve contributed to the project and is our way of thanking those who help with the project.

Thank you, Is it appropriate that just add myself now within this PR since you did not merge yet?

[AntonioMrtz]

Hi @berkaykrc :),
We have CONTRIBUTING and README files that list all the contributors to the project. You can add yourself to both files; this will show that you’ve contributed to the project and is our way of thanking those who help with the project.

Thank you, Is it appropriate that just add myself now within this PR since you did not merge yet?

Yeah, the PR is approved :).

[AntonioMrtz]

Hi @berkaykrc , I think there some missing closing tags in README and CONTRIBUTING :)

[berkaykrc]

Hi @AntonioMrtz,
I think it should be fine now. IDE didn't warned me. I'm little rusty on HTML 😄

[berkaykrc]

I tried to squash all commits into one but looks like couldn't manage it. I needed to git push --force apparently.