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

Update markup.lua #404

Closed

Conversation

alejandro-alzate
Copy link
Contributor

Fixed: /usr/local/share/lua/5.1/ldoc/markup.lua:159: attempt to index local 'line' (a nil value)
By adding a check enclosing all the lines that play with line making sure is not nil

I Don't know exactly why is being triggered but i hope it helps.

@alejandro-alzate
Copy link
Contributor Author

Making a better one following the indentation of the repo

@alerque
Copy link
Member

alerque commented Mar 15, 2024

For future reference next time don't close PRs, just continue making amend commits in the same branch.

@alejandro-alzate
Copy link
Contributor Author

For future reference next time don't close PRs, just continue making amend commits in the same branch.

Please excuse me
I realized that too late, when i did the other one, should i reopen it? if so how i can delete the other one? (#405)

@alerque
Copy link
Member

alerque commented Jul 22, 2024

No worries, you live and learn. Since you already have the other one open (and there is no delete) we'll just close this one and more on with review in the other one.

@alerque alerque closed this Jul 22, 2024
@alejandro-alzate
Copy link
Contributor Author

alejandro-alzate commented Jul 22, 2024

you:

you live and learn

my bio:

open to help and get helped

Yeah last time I teached a translator how to rename a pr.

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

Successfully merging this pull request may close these issues.

2 participants