Skip to content
This repository has been archived by the owner on Oct 28, 2020. It is now read-only.

Fixes #44: Allow named reduce function in the cozy model; #45

Closed
wants to merge 1 commit into from

Conversation

bnjbvr
Copy link
Contributor

@bnjbvr bnjbvr commented Oct 12, 2015

No description provided.

@bnjbvr
Copy link
Contributor Author

bnjbvr commented Oct 12, 2015

@aenario or @frankrousseau, can you get a look at this PR, please? It fixes #44.

@bnjbvr
Copy link
Contributor Author

bnjbvr commented Oct 15, 2015

Well well well, the CI seems to have found a bug, which is a good thing :-)
Any idea why the request creation would work, but not running the actual request? (if you want to take over this PR, feel free)

@frankrousseau
Copy link
Contributor

Thank you @bnjbvr for that fix.

I have to think to understand this line. So, there is something wrong:
"function #{reduce.slice reduce.indexOf '('}"

Could explicit it and tell why you do that in the comments?

aenario pushed a commit that referenced this pull request Oct 19, 2015
@aenario aenario closed this Oct 19, 2015
@aenario
Copy link
Contributor

aenario commented Oct 19, 2015

Pr replaced with fixed #48

frankrousseau pushed a commit that referenced this pull request Oct 19, 2015
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.

3 participants