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.

Sign up for GitHub

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

Jump to bottom

remove dependency on List::MoreUtils #309

Merged
merged 1 commit into from
Mar 27, 2017
Merged

remove dependency on List::MoreUtils #309

merged 1 commit into from
Mar 27, 2017

Conversation

SysPete
Copy link
Contributor

@SysPete SysPete commented Mar 25, 2017

Only any was used so switching to List::Util::any removes a non-core dependency for Perls >= 5.22 (any added to List::Util at 1.33).

If you prefer to simply switch to using grep I'd be happy to provide a replacement PR.

@SysPete
remove dependency on List::MoreUtils
00696a6 
Only `any` was used so switching to List::Util::any removes a non-core
dependency for Perls >= 5.22 (`any` added to List::Util at 1.33).
@gempesaw
Copy link
Collaborator

Hooray for removing non-core dependencies. Thanks!

@gempesaw gempesaw merged commit 3498404 into teodesian:master Mar 27, 2017
gempesaw added a commit that referenced this pull request May 25, 2017
@gempesaw
v1.19
f1370c2 
        [BUG FIXES]
        - Shim use of name as find method for geckodriver
        - Start up geckodriver/binary Firefox properly
        - #309, #310: @SysPete: Fix issues with dependencies

        [NEW FEATURES]
        - #307: @vitstradal: Re-use an existing session
gempesaw added a commit that referenced this pull request May 25, 2017
@gempesaw
v1.20
cbe3e44 
        [BUG FIXES]
        - Shim use of name as find method for geckodriver
        - Start up geckodriver/binary Firefox properly
        - #309, #310: @SysPete: Fix issues with dependencies
        - #313, #312: @lukec: Handle saucelabs redirect during session create

        [NEW FEATURES]
        - #307: @vitstradal: Re-use an existing session
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SysPete @gempesaw