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

Add notes to contributing docs to help initial setup #130

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

OmegaJak
Copy link
Contributor

@OmegaJak OmegaJak commented Oct 4, 2023

Added two notes for things that tripped me up during my initial setup for contribution.

@@ -126,6 +126,8 @@ If you're prepared to contribute by submitting a pull request, ensure you have m
- `cargo check` has been run.
- `cargo sqlx prepare` has been run.

> Note: If you encounter issues with `sqlx` saying 'no queries found' after running `cargo sqlx prepare`, you may need to ensure the installed version of `sqlx-cli` matches the current version of `sqlx` used [in labrinth](https://github.com/modrinth/labrinth/blob/master/Cargo.toml).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure if this is really the proper solution, but it's what got cargo sqlx prepare to work for me (based on launchbadge/sqlx#788 (comment))

@OmegaJak OmegaJak requested a review from triphora October 4, 2023 15:21
@triphora triphora merged commit 9a6b800 into master Oct 4, 2023
1 check passed
@triphora triphora deleted the add_more_initial_setup_guidelines branch October 4, 2023 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants