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
Upgrade scaffold and introduce Github Action #40
Conversation
.github/security.md
Outdated
@@ -1,4 +1,15 @@ | |||
## Security contact information | |||
# Security Policy |
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.
@sttk I added a "global" security policy to the gulpjs github organization, so we can probably remove this file.
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.
Got it. I'll remove this file.
test/index.js
Outdated
|
||
it('should register a module loader for the specified extension with cwd', function(done) { |
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.
New test?
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.
Yes. there was no case about cwd option and this case make coverage to 100%.
@phated I'm sorry for too late reply. I modified what you pointed out. |
I'm going to make some more changes to the scaffold and apply to this repository. I will probably apply them to this PR and then merge it. |
Thanks @sttk! I'm going to be releasing 0.8.0 shortly. I appreciate your work here 🍻 |
I modified this project referring to replace-ext.