solresol / wordnetify-tinystories

Takes a TinyStories file and converts each word to a wordnet node
0 stars 0 forks source link

Sweep: Change .github/workflows/run-wordnetify.yml to be a test #27

Closed solresol closed 1 day ago

solresol commented 1 day ago

Details

.github/workflows/run-wordnetify.yml currently creates a database called sample.sqlite

After that, it should run sqlite3 sample.sqlite .dump > generated-sample.sql

Then it should compare if there are differences between generated-sample.sql and tests/sample.sql

If there are differences, it should error the workflow (i.e. the test failed).

Checklist - [X] Modify `.github/workflows/run-wordnetify.yml` ! No changes made [Edit](https://github.com/solresol/wordnetify-tinystories/edit/sweep/change_githubworkflowsrunwordnetifyyml_1/.github/workflows/run-wordnetify.yml) - [X] Running GitHub Actions for `.github/workflows/run-wordnetify.yml` ✗ [Edit](https://github.com/solresol/wordnetify-tinystories/edit/sweep/change_githubworkflowsrunwordnetifyyml_1/.github/workflows/run-wordnetify.yml)
ifost-autodev[bot] commented 1 day ago
Sweeping

50%

Actions (click)

GitHub Actions✓

Here are the GitHub Actions logs prior to making any changes:

Sandbox logs for bea2998
Checking .github/workflows/run-wordnetify.yml for syntax errors... ✅ .github/workflows/run-wordnetify.yml has no syntax errors! 1/1 ✓
Checking .github/workflows/run-wordnetify.yml for syntax errors...
✅ .github/workflows/run-wordnetify.yml has no syntax errors!

Sandbox passed on the latest main, so sandbox checks will be enabled for this issue.


❌ Unable to Complete PR

I'm sorry, but it looks like an error has occurred due to a planning failure. Feel free to add more details to the issue description so Sweep can better address it. Alternatively, reach out to Kevin or William for help at https://discord.gg/sweep.

For bonus GPT-4 tickets, please report this bug on Discord (tracking ID: f9ddc94270).


Please look at the generated plan. If something looks wrong, please add more details to your issue.

File Path Proposed Changes
.github/workflows/run-wordnetify.yml Modify .github/workflows/run-wordnetify.yml with contents:
• Add a new job to the existing workflow that sets up a Python environment with necessary dependencies as specified in README.md.
• Include a step to create a SQLite database named sample.sqlite using the schema and data from tests/sample.sql.
• Add a command to dump the contents of sample.sqlite into a new file named generated-sample.sql using sqlite3 sample.sqlite .dump > generated-sample.sql.
• Introduce a step to compare generated-sample.sql with tests/sample.sql using a diff command.
• Implement error handling such that if the diff command finds differences, the workflow should exit with a non-zero status, causing the test to fail.
• Ensure the workflow cleans up any generated files after the test to maintain a clean environment for subsequent runs.

🎉 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.