-
Notifications
You must be signed in to change notification settings - Fork 766
Visual Prolog programming language #2480
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
CI is failing in |
Co-authored-by: Jean Abou-Samra <[email protected]>
|
Common Lisp is not my language, but I believe that '^\s*(defun\s' will be a very good indicator for that language. I can add the rule if you like. |
|
Yes, good. |
|
Fixed |
|
Please take a look at the test failures revealed by the CI. |
|
Yes, thank you for your patience :-). I have successfully run 'py -m tox" now. |
|
Still failing CI. Can you try to run |
|
tox is not a stand alone command on my computer. And it does not appear to have a -e option. Running py -m tox -- -c check says: ... ========================================== 4140 passed, 12 skipped in 19.35s ========================================== Maybe it is one of the 12 skipped tests that should have failed. |
|
Generated new _mapping.py files. Which unfortunately get line endings (and have to be corrected manually afterwards). Anyways, now the out put from py -m tox - e check is: $ py -m tox -e check And those errors are meaningless to me. |
|
The tox command leaves 70 python.exe processes running. Not being able to figure out how to run the "check" I have instead added a doc string with copyright. And will (sorry) waste a couple of your trees. |
|
I was able to run 'py -m tox -e check' on a computer with fewer processors (I think the number of processors makes influences the number of python processes that are spawned and 64 processors is probably too many (there was something about too many handles in the errors). However on that computer the -e check succeeds even though it failed when I pushed it to git. Tried once more hope this time will succeed (I hoped that the other times as well). |
|
Can you check the "Allow edits and access to secrets by maintainers" checkbox in the right sidebar please? That way we can fix those issues for you. |
|
Oh, actually I see that it's checked. Which is strange because |
|
I am blank about the permission thing. Also my test run says no errors: check: commands[2]> flake8 --select F --ignore F401 pygments Anyways I figured out that the underlining was wrong length, and have fixed that also. |
|
And finally the testsuite succeeded :-). |
|
Is there any problems I have to deal with? |
|
I'm on vacation right now, please allow a day or two.
Le lundi 28 août 2023 à 03:43 -0700, Thomas Linder Puls a écrit :
… Is there any problems I have to deal with?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID:
***@***.***>
|
|
Of, course. Have a nice vacation. |
|
Thank you. |
Added Visual Prolog language (https://www.visual-prolog.com).