Skip to content

Adds stub definition to the ETL exercise#550

Merged
coriolinus merged 2 commits intoexercism:masterfrom
tushartyagi:master
Jun 22, 2018
Merged

Adds stub definition to the ETL exercise#550
coriolinus merged 2 commits intoexercism:masterfrom
tushartyagi:master

Conversation

@tushartyagi
Copy link
Copy Markdown
Contributor

The lib.rs file of ETL exercise is missing the function stub. Adding it in this commit.

Comment thread exercises/etl/src/lib.rs
use std::collections::BTreeMap;

pub fn transform(h: &BTreeMap<i32, Vec<char>>) -> BTreeMap<char, i32> {
unimplemented!("How will you transform the tree {:?}?", h)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please express this in the form of an imperative statement: "Transform this tree: {:?}" or similar.

Copy link
Copy Markdown
Member

@coriolinus coriolinus left a comment

Choose a reason for hiding this comment

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

Thanks @tushartyagi for this PR! It looks good to me.

I've requested one change, but it is optional.

I'm going to leave this for a while to ensure other maintainers have a chance to see it. Assuming there are no objections, I intend to merge this on Friday the 22nd. If on Saturday I still have not done so, please feel free to ping me a reminder.

@coriolinus coriolinus merged commit e9e2abc into exercism:master Jun 22, 2018
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.

3 participants