Skip to content
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

probando #25

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

probando #25

wants to merge 9 commits into from

Conversation

pfuentes2021
Copy link

No description provided.

@apiad
Copy link
Member

apiad commented Sep 25, 2021

Fijate que todos los errores que da son por el formato de la salida de error. Revisalo un momento porque es posible que sea una bobería, a ver si logramos que ejecuten los tests. Si no lo logras me dices.

@apiad
Copy link
Member

apiad commented Sep 26, 2021

Creo que el error que está dando ahora es que cuando hay error de compilación debes salir con código de retorno 1, o sea exit(1) en vez de exit()

src/main.py Outdated
lexer = MyLexer(input)
lexer.build()
if lexer.check():
exit()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aquí por ejemplo

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

Successfully merging this pull request may close these issues.

3 participants