Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated TUTORIAL.md with an additional instructional step and Troubleshooting Section #1104

Merged
merged 3 commits into from
May 18, 2024

Conversation

dianna-SE
Copy link
Contributor

I added an extra step for installing a package as this part was missing. It might be intuitive but some users may forget and would encounter an error. This step was included in README but not in TUTORIAL. A Troubleshooting section was added to address some common issues with installations. Changes were updated to CHANGELOG.md.

dianna-SE and others added 3 commits May 2, 2024 17:47
@dieser-niko
Copy link
Member

I think you managed to commit something that isn't related to the description of your pull request

@dianna-SE
Copy link
Contributor Author

Thanks for the catch! Removed the wrong commits from the PR and made another branch for the other work.

@dieser-niko
Copy link
Member

I'd recommend doing it the other way around, but that's up to you. As for the changes themselves, they look fine and it makes sense to include some tips in the tutorial.

Copy link
Member

@dieser-niko dieser-niko left a comment

Choose a reason for hiding this comment

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

Some more tips would be appreciated, but maybe in the future.

@dieser-niko dieser-niko merged commit 74330da into spotipy-dev:master May 18, 2024
12 checks passed
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.

None yet

2 participants