-
Notifications
You must be signed in to change notification settings - Fork 352
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
0.34 rc #160
0.34 rc #160
Conversation
|
||
# @param [Pathname] repos_dir @see repos_dir. | ||
# @param [<Array<Pathname] repos_dirs @see directories |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be Array<Pathname>
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you are referring to the angled brackets... definitely!
047e7aa
to
b69df74
Compare
b69df74
to
9d98426
Compare
/cc @alloy @segiddins are we happy with this one? |
def dependencies_to_lock_pod_named(name) | ||
result = [] | ||
deps = dependencies.select { |d| d.root_name == name } | ||
if deps.empty? | ||
raise StandardError, "Attempt to lock the `#{name}` Pod without an " \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Attempted
or Attempting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a
not an
Merging for the sake of getting RC1 out. |
This PR supersedes #19. The commits of @netbe have been squashed and reworded.