This repository has been archived by the owner. It is now read-only.

Switch template instructions #195

Merged
merged 2 commits into from Jun 15, 2014

Conversation

Projects
None yet
4 participants
@fgallaire
Contributor

fgallaire commented Dec 11, 2013

The form is:

{'case', SwitchVariable :: atom(), Values :: list(), Instructions :: list() | tuple()}.

This completes #138 "Support conditional template instructions".

@tuncer

This comment has been minimized.

Show comment
Hide comment
@tuncer

tuncer Jan 11, 2014

Contributor

cc @nox

Contributor

tuncer commented Jan 11, 2014

cc @nox

@jaredmorrow

This comment has been minimized.

Show comment
Hide comment
@jaredmorrow

jaredmorrow Mar 5, 2014

Contributor

Can we get an example and a test added for this feature?

Contributor

jaredmorrow commented Mar 5, 2014

Can we get an example and a test added for this feature?

@fgallaire

This comment has been minimized.

Show comment
Hide comment
@fgallaire

fgallaire Mar 6, 2014

Contributor

Hello,

I developed this feature because it was useful to me for that project:
https://github.com/fgallaire/colt-templates

As the "conditional template instruction" doesn't have test, I don't know exactly what to do.

Contributor

fgallaire commented Mar 6, 2014

Hello,

I developed this feature because it was useful to me for that project:
https://github.com/fgallaire/colt-templates

As the "conditional template instruction" doesn't have test, I don't know exactly what to do.

@tuncer

This comment has been minimized.

Show comment
Hide comment
@tuncer

tuncer May 29, 2014

Contributor

What's the status?

Contributor

tuncer commented May 29, 2014

What's the status?

@tsloughter tsloughter closed this Jun 15, 2014

@fgallaire

This comment has been minimized.

Show comment
Hide comment
@fgallaire

fgallaire Jun 15, 2014

Contributor

Few lines of code for a (for some people) useful option, what is the problem ?

Contributor

fgallaire commented Jun 15, 2014

Few lines of code for a (for some people) useful option, what is the problem ?

@tsloughter

This comment has been minimized.

Show comment
Hide comment
@tsloughter

tsloughter Jun 15, 2014

Member

Just trying to clean up and sort of start fresh. Closed everything that was opened before March of this year.

But, this looks fine. I think I'll reopen it and merge in.

Member

tsloughter commented Jun 15, 2014

Just trying to clean up and sort of start fresh. Closed everything that was opened before March of this year.

But, this looks fine. I think I'll reopen it and merge in.

@fgallaire

This comment has been minimized.

Show comment
Hide comment
@fgallaire

fgallaire Jun 15, 2014

Contributor

Thanks a lot @tsloughter

Contributor

fgallaire commented Jun 15, 2014

Thanks a lot @tsloughter

@tsloughter tsloughter reopened this Jun 15, 2014

tsloughter added a commit that referenced this pull request Jun 15, 2014

@tsloughter tsloughter merged commit 29a16cb into rebar:master Jun 15, 2014

1 of 2 checks passed

continuous-integration/travis-ci The Travis CI build is in progress
Details
default The Travis CI build passed
Details
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.