Skip to content

Conversation

@Zamiell
Copy link
Contributor

@Zamiell Zamiell commented Feb 22, 2023

No description provided.

Copy link
Member

@Perryvw Perryvw left a comment

Choose a reason for hiding this comment

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

I really wish you had split these edits up over multiple MRs, one per file, and had not changed half the website in a MR called 'update external-code.md'

@Zamiell Zamiell changed the title Update external-lua-code.md Rewrite some sections of the website to make it more comprehensive Feb 24, 2023
@Zamiell
Copy link
Contributor Author

Zamiell commented Feb 24, 2023

i think i fixed all the issues now

Copy link
Member

@Perryvw Perryvw left a comment

Choose a reason for hiding this comment

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

I don't know why I have to ask this again, but please DO NOT rename docs files, it will break external links pointing to our documentaiton.

@Zamiell
Copy link
Contributor Author

Zamiell commented Feb 28, 2023

i figured it was unlikely for there to be external links to these pages specifically, and it makes the new organization look less bugged.
but regardless it should be fixed now.

@Perryvw Perryvw merged commit bbb2749 into TypeScriptToLua:source Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants