Skip to content
This repository has been archived by the owner on May 2, 2022. It is now read-only.

Rewriting index.coffee.. #87

Open
wants to merge 5 commits into
base: next
Choose a base branch
from
Open

Conversation

rlindsberg
Copy link
Collaborator

  • Add script to build ES6 with babel
  • Import module os and list network interfaces

@rlindsberg rlindsberg changed the base branch from master to next August 27, 2018 19:07
@rlindsberg
Copy link
Collaborator Author

@alexkirsz Hi! Can you check it out if I am doing it right?

@alexkirsz
Copy link
Owner

@rlindsberg This is fine :)

On the CLI side, I'd recommend reimplementing the current behavior for now. You can consider one PR to be one feature, to be merged once the implementation is complete.

@rlindsberg
Copy link
Collaborator Author

@alexkirsz Which feature should I work on first? I feel kinda difficult to find somewhere to begin with..

@alexkirsz
Copy link
Owner

@rlindsberg For now, the focus is on translating the codebase from CoffeeScript to JavaScript. So it doesn't really matter which feature you implement first, as long as the new code is functionally equivalent to the old code.

@alexkirsz
Copy link
Owner

So I'd say just start with a file (or parts of a file) and translate it 1:1, make a PR and I'll check it out :)

@rlindsberg rlindsberg changed the title Little progress implementing babel.. Rewriting index.coffee.. Jul 13, 2019
@rlindsberg rlindsberg requested a review from alexkirsz July 13, 2019 20:05
@rlindsberg rlindsberg self-assigned this Jul 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants