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

replace map expressions that aren't doing list transformations #33

Merged
merged 1 commit into from Feb 12, 2016

Conversation

Projects
None yet
2 participants
@mcmillhj
Contributor

mcmillhj commented Feb 12, 2016

patch for issue #30, remove map statements that don't produce a resulting list in favor of foreach loops

replace map expressions that aren't doing list transformations
patch for issue#30, remove map statements that don't produce a resulting list in favor of foreach loops
@mcmillhj

This comment has been minimized.

Show comment
Hide comment
@mcmillhj

mcmillhj Feb 12, 2016

Contributor

Ok, clean slate. Promise I won't mangle this one :)

Contributor

mcmillhj commented Feb 12, 2016

Ok, clean slate. Promise I won't mangle this one :)

@SysPete

This comment has been minimized.

Show comment
Hide comment
@SysPete

SysPete Feb 12, 2016

Member

Many thanks @mcmillhj - merging this one. Closes #30

Member

SysPete commented Feb 12, 2016

Many thanks @mcmillhj - merging this one. Closes #30

SysPete added a commit that referenced this pull request Feb 12, 2016

Merge pull request #33 from mcmillhj/enhancement/no-map-in-void-context2
replace map expressions that aren't doing list transformations

@SysPete SysPete merged commit fd071d0 into interchange:master Feb 12, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@mcmillhj mcmillhj deleted the mcmillhj:enhancement/no-map-in-void-context2 branch Feb 12, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment