Skip to content

Conversation

waldyrious
Copy link

@waldyrious waldyrious commented Sep 14, 2017

Not sure why the diff shows whitespace changes; I didn't touch any other line, so this was probably due to github's editor standardizing the indentation throughout the file.

In any case, the clean diff can be seen here.

@cdrnet
Copy link
Member

cdrnet commented Sep 15, 2017

Thanks!

I do wonder, isn't this a generated file? Maybe this would need to be fixed also in the original source. I haven't touched this for a while...

@waldyrious
Copy link
Author

Yeah, I thought that too, but don't know the doc generation pipeline. The only other place this sentence appears, as far as I can tell, is in Library/Quaternion.cs, but it seems to be fine there.

@waldyrious
Copy link
Author

@cdrnet so how to we move forward here? I suppose even if this is a generated file, there's no harm in merging the PR. If you'd prefer me to proceed some other way, please let me know.

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