Skip to content

Conversation

@gwpl
Copy link

@gwpl gwpl commented Nov 28, 2023

parsers.py += .sw Sway syntax # https://fuellabs.github.io/sway

Tested using python setup.py docker_test from #1 .
However test is failing for some reason , and my python-fu is too weak to fit it....

Tree sitter Sway implementation: https://github.com/FuelLabs/tree-sitter-sway

Greg added 2 commits November 28, 2023 14:10
This commit introduces a new commands in setup.py that allows users to build the Docker image and run pytest tests inside it.

The README.md file has been updated to document this new feature, providing clear instructions for users to take advantage of reproducible testing environments.

* `python setup.py build_docker` to facilitate Docker image building.
* `python setup.py docker_test` for running pytest tests within the Docker container.
* Enhanced `README.md` with instructions on how to use the new `python setup.py build_docker` and `python setup.py docker_test` commands.
* The new Docker-related setup commands ensure consistent testing environments across different systems.
* These additions help maintain uniformity in test execution and assist in the debugging process for developers and contributors.
Tested using `python setup.py docker_test` from Aider-AI#1 .

Tree sitter Sway implementation: https://github.com/FuelLabs/tree-sitter-sway
@gwpl gwpl mentioned this pull request Nov 30, 2023
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Greg seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants