Skip to content

Comments

Expand documentation of git dependency subkeys#1802

Merged
finswimmer merged 1 commit intopython-poetry:masterfrom
rpdelaney:docs/dependency_rev_pin
Dec 29, 2019
Merged

Expand documentation of git dependency subkeys#1802
finswimmer merged 1 commit intopython-poetry:masterfrom
rpdelaney:docs/dependency_rev_pin

Conversation

@rpdelaney
Copy link
Contributor

I need to pin a dependency to a specific git commit hash. After reading
these docs, it was not clear to me that I could do so with the rev
key, because "something else" is vague. I wound up digging into the
source code and learned what I needed there.

This change expands the documentation for git dependencies to explain
the sub-keys in more detail and give examples of how to use each.

I need to pin a dependency to a specific git commit hash. After reading
these docs, it was not clear to me that I could do so with the `rev`
key, because "something else" is vague. I wound up digging into the
source code and learned what I needed there.

This change expands the documentation for `git` dependencies to explain
the sub-keys in more detail and give examples of how to use each.
Copy link
Member

@finswimmer finswimmer left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your contribution!

@finswimmer finswimmer merged commit 0bbeae3 into python-poetry:master Dec 29, 2019
@rpdelaney rpdelaney deleted the docs/dependency_rev_pin branch February 25, 2020 21:00
@github-actions
Copy link

github-actions bot commented Mar 1, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants