Skip to content

Conversation

@houli
Copy link
Contributor

@houli houli commented Oct 16, 2017

Mostly followed the current exercises, in particular this PR #44. I see that some of the config.json keys have changed since that pull request so this will probably need some changes there.

@houli houli force-pushed the rna-transcription branch from 5280c50 to e7afdb1 Compare October 16, 2017 21:02
@paf31
Copy link
Contributor

paf31 commented Oct 19, 2017

Hmm looks like Travis is timing out. @lpil @icyrockcom Any ideas?

I'll restart it for now.

"output"
],
"dependencies": {
"purescript-console": "^3.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you need all of these.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, not sure what the deal is with the individual bower json files, the README isn't so clear. All of the exercises seem to have the same set of dependencies

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe all the exercises share the same deps now so that we can cache easier on CI

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh....

Ok, ignore me then.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@houli Can you open an issue with details about what was unclear in the README, so I submit a PR to clear things up for future contributors?

@paf31 Yes, recently changed, details in: #71, #72 and #73

import Data.Traversable (traverse)

toRNA :: String -> Maybe String
toRNA dna = fromCharArray <$> traverse complement (toCharArray dna)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Copy link
Contributor

@paf31 paf31 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, pending CI passing, and minimizing the dependencies please. Thanks!

@paf31
Copy link
Contributor

paf31 commented Oct 20, 2017

If you could please resolve conflicts here, I'll get this merged. Thanks!

@paf31 paf31 merged commit 56c0f09 into exercism:master Oct 21, 2017
@paf31
Copy link
Contributor

paf31 commented Oct 21, 2017

Thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants