Skip to content

Conversation

@tejasbubane
Copy link
Member

@tejasbubane tejasbubane commented Mar 22, 2019

Problem spec PR: exercism/problem-specifications#1466

Closes #641

@SleeplessByte
Copy link
Member

Looks fine on first skim.

Don't merge ⚠️ yet -- Maud and I are figuring out if we want to combine #639 #640 #641 in one go.

Copy link
Member

@SleeplessByte SleeplessByte left a comment

Choose a reason for hiding this comment

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

See inline comment

@SleeplessByte
Copy link
Member

Discussed this with Maud. We'll be merging #640 + #639 + #641 simultaneously as a track anatomy experiment.

@SleeplessByte
Copy link
Member

Don't forget to update and copy

  • package.json
  • babel.config.js
  • .eslintrc

@emcoding
Copy link

Fantastic!

@tejasbubane
Copy link
Member Author

@SleeplessByte Done

@SleeplessByte SleeplessByte merged commit c1d9030 into exercism:master Mar 26, 2019
@emcoding
Copy link

Yay!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants