Skip to content

default to "HEAD" revision#287

Closed
djeebus wants to merge 6 commits into
jacebrowning:mainfrom
djeebus:default-to-HEAD
Closed

default to "HEAD" revision#287
djeebus wants to merge 6 commits into
jacebrowning:mainfrom
djeebus:default-to-HEAD

Conversation

@djeebus

@djeebus djeebus commented Sep 9, 2022

Copy link
Copy Markdown

Not every repo has a main or master, but they all have a HEAD.
Also make sure that every Source has a rev

The changes to __post_init__ also have the happy side effect of removing the need to define a rev if the source should point to main/master.

not every repo has a main or master, but they all have a HEAD
make sure that every Source has a rev
@djeebus
djeebus force-pushed the default-to-HEAD branch 2 times, most recently from 6922fde to 18180d1 Compare September 9, 2022 15:37
@jacebrowning

Copy link
Copy Markdown
Owner

Thanks for the PR! I haven't seen anyone else report rev not working so I don't plan to accept it at this time.

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