New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
working and passing tests #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added a linter. There are a few things to clean up. I'll reach out about general code review as well.
index.js
Outdated
reject | ||
}); | ||
if (!working) { | ||
one(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding top down like this it would be helpful to have a more descriptive name on this one.
index.js
Outdated
} | ||
function createWorker() { | ||
const worker = cp.fork(`${__dirname}/worker.js`, { | ||
stdio: 'inherit' // TODO back to ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the work to be done here?
Hi @abea! I opened this PR on a new module properly! I'm so goddamn excited!
What do you think? The only thing I should have to tell you, if I wrote the README properly, is that this is a DGAD functional requirement; they need a forms widget that allows editors to supply their own (potentially slow) regular expressions for form validation.