deep-foundation / loggify-js

Experiment
The Unlicense
0 stars 2 forks source link

Initialize repository for loggify-js #1

Closed Konard closed 1 year ago

Konard commented 1 year ago

loggify-js is a cli tool that should be written in javascript.

This tool should be able to add console.log statement after each statement in JavaScript file.

The tool should be installed globally and executed like this:

loggify-js index.js

The result should be an updated in-place index.js file that has additional console.log statements after each statement in each funciton.

Checklist - [X] `package.json` ✅ Commit [`d80e9bf`](https://github.com/deep-foundation/loggify-js/commit/d80e9bffb4506b80308f86f5cb16f4d06131e865) - [X] `index.js` ✅ Commit [`7b15b01`](https://github.com/deep-foundation/loggify-js/commit/7b15b011ef98649a19145dcab959917a85dde9ac) - [X] `.gitignore` ❌ Failed
deep-foundation-sweepai[bot] commented 1 year ago

Here's the PR! https://github.com/deep-foundation/loggify-js/pull/2.

💎 Sweep Pro: I'm creating this ticket using GPT-4. You have unlimited GPT-4 tickets.

Actions (click)


Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/deep-foundation/loggify-js/blob/307017d72bcbc987742ab22e91fc451f306bf87b/README.md#L1-L7

Step 2: ⌨️ Coding


Step 3: 🔁 Code Review

Here are my self-reviews of my changes at sweep/initialize-repo.

Here is the 1st review

Hi there, Great job on your first pull request! There's just one small issue that needs to be addressed. In the `index.js` file, the `addLogs` function is currently adding `console.log` statements after each expression, not each statement as required by the issue. You can fix this by checking if the node type is 'Statement' instead of 'ExpressionStatement'. Here's the line that needs to be changed: - Change required in `index.js` on line 12 Keep up the good work!

I finished incorporating these changes.


🎉 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

Konard commented 1 year ago

This issue is archived.