Abdur-rahmaanJ / greenberry

A one-line statement language with grammar and infinite look-ahead
https://abdur-rahmaanj.github.io/greenberry/
Apache License 2.0
50 stars 39 forks source link

Fix run error #80

Open Abdur-rahmaanJ opened 1 year ago

Abdur-rahmaanJ commented 1 year ago
0xGodspeed commented 1 year ago

Hey, can you give more details about the errors? I did python -m pip install greenberry and it gave me no errors. Running a basic .gb file doesn't give any errors either.

Abdur-rahmaanJ commented 1 year ago

@0xGodspeed output does not correspond with instructions

0xGodspeed commented 1 year ago

I am really sorry but I still can't find the error. Could you please tell me what command gives the wrong output? image

Abdur-rahmaanJ commented 1 year ago

@0xGodspeed Sorry you must install green berry using python -m pip install -e .

0xGodspeed commented 1 year ago

I believe this is the problem https://github.com/Abdur-rahmaanJ/greenberry/blob/a94e6ba2dd4836bad6b883acd8cd775c5a71a6c0/setup.py#L90 And this seems to fix it:

 packages=find_packages('src'), 

I'm not exactly sure how setuptools works and why it searches for the tests directory in src/ without this. Please explain to me if you can.

Abdur-rahmaanJ commented 1 year ago

@0xGodspeed You have an interesting problem which this issue does not describe, how did you encounter the test error?

0xGodspeed commented 1 year ago

by doing python -m pip install -e

Abdur-rahmaanJ commented 1 year ago

@0xGodspeed Ok open a PR to fix it!

0xGodspeed commented 1 year ago

@Abdur-rahmaanJ Is it unrelated from this issue? Should I mention this issue in the PR?

Abdur-rahmaanJ commented 1 year ago

Yes, just add an orphan PR

BernardVos commented 1 year ago

Hi all,

I'd like to contribute to this project, but noticed that the last issue is added a year ago. Is this project still alive?

Abdur-rahmaanJ commented 11 months ago

@BernardVos Well, i am all the time available, if you need anything, ping me. If you need a Google meet, mail me: arj.python at gmail.com.