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

Lint manual #25312

Merged
merged 2 commits into from
Jan 23, 2023
Merged

Lint manual #25312

merged 2 commits into from
Jan 23, 2023

Conversation

LeviPesin
Copy link
Contributor

Related issue: #25251

Description

Fix manual so that it adheres to three.js code lint standards.

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@Mugen87
Copy link
Collaborator

Mugen87 commented Jan 21, 2023

I guess the CodeQL warnings are existing ones which just pop up because you are modifying the respective lines of code, right?

@LeviPesin
Copy link
Contributor Author

Yes, I think so.

@Mugen87 Mugen87 added this to the r149 milestone Jan 22, 2023
@Mugen87
Copy link
Collaborator

Mugen87 commented Jan 23, 2023

BTW: Do you know why I can't test your branch via githack? It works with other PRs without issues.

So this link should work: https://raw.githack.com/LeviPesin/three.js/lint-manual/manual/index.html

@Mugen87 Mugen87 merged commit cdf48ce into mrdoob:dev Jan 23, 2023
@LeviPesin
Copy link
Contributor Author

So this link should work: https://raw.githack.com/LeviPesin/three.js/lint-manual/manual/index.html

I think I made an error while cloning the repository some time ago, so it is actually named LeviPesin/three.js-1, not LeviPesin/three.js...

@LeviPesin LeviPesin deleted the lint-manual branch January 23, 2023 09:39
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