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

docs: fix small typos in the website #7

Merged
merged 4 commits into from
Apr 25, 2024
Merged

docs: fix small typos in the website #7

merged 4 commits into from
Apr 25, 2024

Conversation

colinscz
Copy link
Contributor

@colinscz colinscz commented Mar 3, 2024

Very small adjustments that I found when checking out the website :)

related to #4

Hope it helps 🙏🏽

Copy link

netlify bot commented Mar 3, 2024

‼️ Deploy request for tresjs-web rejected.

Name Link
🔨 Latest commit c13cfb7

@colinscz
Copy link
Contributor Author

@andretchen0 fixed typos, in case there are others I missed please let me know.

@andretchen0 andretchen0 changed the title Fix: fix small typos in the website docs: fix small typos in the website Mar 12, 2024
@andretchen0
Copy link
Contributor

andretchen0 commented Mar 12, 2024

@colinscz

Looks good!

(Just FYI, I changed the PR title from fix: [...] to docs: [...], per the conventional commits specification. fix is reserved for bug patches.)

@alvarosabu

This PR has a couple of changes to the docs. They look good to me.

I haven't been merging PRs here. Want to merge?

@colinscz
Copy link
Contributor Author

@colinscz

Looks good!

(Just FYI, I changed the PR title from fix: [...] to docs: [...], per the conventional commits specification. fix is reserved for bug patches.)

@alvarosabu

This PR has a couple of changes to the docs. They look good to me.

I haven't been merging PRs here. Want to merge?

Thank you, you are absolutely right I missed that for this PR. I presume this will be adjusted with the squash commit by you guys for this time. I keep it in mind for the next one. 💪🏽

@alvarosabu alvarosabu merged commit 74a181f into Tresjs:main Apr 25, 2024
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.

3 participants