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

[developer] Getting Started with Typescript - first pass #96

Merged
merged 4 commits into from
Dec 28, 2023

Conversation

anselm
Copy link
Contributor

@anselm anselm commented Dec 27, 2023

No description provided.

@anselm anselm changed the title note re segue typescript first pass Dec 27, 2023
…{ something } is treated as some kind of variable and should not be used otherwise
@HexaField HexaField merged commit 43f8f7d into main Dec 28, 2023
2 checks passed
@HexaField HexaField deleted the tut-typescript branch December 28, 2023 07:12
@heysokam heysokam changed the title typescript first pass [developer] Getting Started with Typescript - first pass Dec 28, 2023
@heysokam heysokam restored the tut-typescript branch December 28, 2023 19:13
@heysokam
Copy link
Contributor

heysokam commented Dec 28, 2023

This PR breaks the markdown styling guides defined at:
https://github.com/EtherealEngine/etherealengine-docs/?tab=readme-ov-file#contributing-guidelines-documentation-repository
There are some guidelines that I've been following that I was going to review and also annotate in the guidelines today (they are not there listed yet), but didn't have the time to comment on before this merge.

I will restore the branch and open a new PR with the fixes.

@heysokam heysokam added documentation Improvements or additions to documentation developer Anything related to the Developers section of the documentation website labels Dec 28, 2023
@HexaField
Copy link
Member

This PR breaks the markdown styling guides defined at:
https://github.com/EtherealEngine/etherealengine-docs/?tab=readme-ov-file#contributing-guidelines-documentation-repository
There are some guidelines that I've been following that I was going to review and also annotate in the guidelines today (they are not there listed yet), but didn't have the time to comment on before this merge.

I will restore the branch and open a new PR with the fixes.

I merged this in as our CICD had broken and this was passing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer Anything related to the Developers section of the documentation website documentation Improvements or additions to documentation verified-contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants