Closed Konard closed 1 year ago
I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.
[X] src/index.js
✅ Commit 9de33b3
• Import the `process` module at the top of the file • Add a check at the start of the script to see if `process.argv[2]` exists • If `process.argv[2]` exists, call the `loggify` function with `process.argv[2]` as the argument
[X] README.md
✅ Commit 1abb73f
• Add a new section titled "Usage" • In the "Usage" section, explain that the `loggify-js` command can now be used with a file argument like so: `loggify-js` • Explain that if a file argument is provided, the `loggify` function will be executed with it
I have finished reviewing the code for completeness. I did not find errors for sweep/update-index-js
.
.
🎉 Latest improvements to Sweep:
💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. Join Our Discord
File was added instead and not updated.
Update
index.js
to receive argument with file from terminal.And if an argument is received, the
loggify
function should be executed with it.Update
README.md
to informs users the correct way of using the global command.Checklist
- [X] `src/index.js` ✅ Commit [`9de33b3`](https://github.com/deep-foundation/loggify-js/commit/9de33b38d96cda542afe51dae1e4d9d0b1e54358) - [X] `README.md` ✅ Commit [`1abb73f`](https://github.com/deep-foundation/loggify-js/commit/1abb73f12d15d970b33993b52c60d082adf9d674)