Skip to content

Update README.md#69

Merged
rachel-mack merged 2 commits into
masterfrom
install-note
Nov 7, 2023
Merged

Update README.md#69
rachel-mack merged 2 commits into
masterfrom
install-note

Conversation

@rachel-mack
Copy link
Copy Markdown
Contributor

Add note about changing permissions to install poetry - I'm sure how much detail I should add.

e.g. In order to view the folder in Finder, I had to use the Reveal in Finder option from VS code.

Add note about changing permissions to install poetry
Comment thread README.md Outdated
2. Build a wheel using `poetry build`.

> [!NOTE]
> You may need to add read privaleges to your *Users/\<your-username>/.pyenv/* folder.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good! Total nitpick, but can we spell it "privileges" just so no one gets confused about if we're referencing a python library or anything else. Also, is there a reason there's a backslash (\) before <your-username>?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

If you take a look at the README on the branch you can see that the backslash doesn't show up - it's just a necessary escape character within the brackets.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ah whoops, great thanks!

@rachel-mack rachel-mack requested a review from mmeigs November 7, 2023 15:23
@rachel-mack rachel-mack merged commit 5fdf3ef into master Nov 7, 2023
@rachel-mack rachel-mack deleted the install-note branch November 7, 2023 15:27
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.

2 participants