search

Samagra-Development / WarpSQL

WarpSQL
https://samagra-development.github.io/WarpSQL/
Apache License 2.0
16 stars 28 forks source link

Extrack Smoke Tests to a new file #55

Open ChakshuGautam opened 1 year ago

ChakshuGautam commented 1 year ago

Currently, smoke tests are part of the workflow which can lead to a issues in parsing when updating it.

Create a .sql file for smoke tests and add a husky pre-commit and test if it's a valid SQL file.

i-m-karanagarwal commented 1 year ago

Hey @ChakshuGautam , How do I run the pip or npm commands in the codespace to install the requirements? I have added the python externsion but after running the python --version command, it shows python command not found

ChakshuGautam commented 1 year ago

Hey, are you using npm or Python to verify the SQL file?

i-m-karanagarwal commented 1 year ago
image

I tried both the command but none worked.

ChakshuGautam commented 1 year ago

Hey, this project is not a python/npm-based project. What are you using those for? I think those have been wrongly added in this case to setup the codespaces. Please use Gitpod if you are working on this. I will fix those docs in the meanwhile.

i-m-karanagarwal commented 1 year ago

Okay @ChakshuGautam ,I will try this in Gitpod.

i-m-karanagarwal commented 1 year ago
image

I tried to set up the env using gitpod but I am facing this error. What should I do to resolve it?

Amrittttt commented 1 year ago

Can you assign this for me?

Amrittttt commented 1 year ago

@ChakshuGautam

PiyushRaj927 commented 1 year ago
image

I tried to set up the env using gitpod but I am facing this error. What should I do to resolve it?

Hi @i-m-karanagarwal, this is a known issue which I have addressed in PR #54. While the PR is being reviewed, you can manually update your local copy to continue development.