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

Fix a typo in multiple places #24195

Merged
merged 2 commits into from
Jun 7, 2022
Merged

Fix a typo in multiple places #24195

merged 2 commits into from
Jun 7, 2022

Conversation

Dannie226
Copy link
Contributor

Related issue: Was not put into a Github Issue. Issue is on three.js discourse

Description

There was a typo in every single three.js file, and the three.js source file. The typo was where in a few places, instead of "interleaved" the files said "interlaved." This pull request will fix that.

@LeviPesin
Copy link
Contributor

Please remove build files from this PR.

@Dannie226
Copy link
Contributor Author

The typo was also in the build files. Why remove them?

@LeviPesin
Copy link
Contributor

Because they are auto-built from the source files.

@Dannie226
Copy link
Contributor Author

How would I remove the files from the PR? I changed the typos all in one commit, so I don't know how I could remove the files without deleting the pull request and starting over.

@LeviPesin
Copy link
Contributor

Push the second commit to the branch.

@Dannie226
Copy link
Contributor Author

What second commit? I only made one commit, and it changed the typo in all 5 places.

@LeviPesin
Copy link
Contributor

Make a new, second commit restoring the build files and just push it to the branch.

Since the build files are auto-generated from the source, the typo
should only have fixed it in the src folder. This commit fixes that
@Dannie226
Copy link
Contributor Author

Sorry for the trouble. I am basically github illiterate. I really only use it for storing code, so I don't really know how it works past git command line stuff for pushing and pulling.

@mrdoob mrdoob added this to the r142 milestone Jun 7, 2022
@mrdoob mrdoob merged commit 7da24e3 into mrdoob:dev Jun 7, 2022
@mrdoob
Copy link
Owner

mrdoob commented Jun 7, 2022

Thanks!

abernier pushed a commit to abernier/three.js that referenced this pull request Sep 16, 2022
* Fix a typo in multiple places

* Undo typo fix in build files

Since the build files are auto-generated from the source, the typo
should only have fixed it in the src folder. This commit fixes that

Co-authored-by: Carl Lee Landskron <[email protected]>
snagy pushed a commit to snagy/three.js-1 that referenced this pull request Sep 21, 2022
* Fix a typo in multiple places

* Undo typo fix in build files

Since the build files are auto-generated from the source, the typo
should only have fixed it in the src folder. This commit fixes that

Co-authored-by: Carl Lee Landskron <[email protected]>
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.

4 participants