check-it-out
check-it-out copied to clipboard
chore(deps): remove danger
remove danger and its dependencies
✅ Closes: #493
Description
Uninstalled danger package removed config files related to danger - dangerfile.js, dangerfile.js.sample removed a job from Travis config that used danger package ref #493
Types of changes
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] Documentation update
- [x] Refactor
Checklist:
- [x] I have read the CONTRIBUTING document.
- [x] My commits follow the commitizen commit convention
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
This pull request fixes 1 alert when merging 29bca12750494413bb6659485d9a58d4599edde4 into 000f416121c6e96e511d909c4ec1e4e123d01261 - view on LGTM.com
fixed alerts:
- 1 for Unused variable, import, function or class
hello, @jwu910 I can review this if needed
Thanks @a-camarillo great point.
Been a while since I looked at this project and chore does make more sense for this removal.
Hi @KalaRaja I see we've changed the title, but can you please update your commit message as well please?
If you need help, theres lots of info at this article.
Any update on this? May close this and bump the ticket back to todo soon
Stale