Skip to content

fixing documents #3072

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

Closed
wants to merge 7 commits into from
Closed

fixing documents #3072

wants to merge 7 commits into from

Conversation

c-chaitanya
Copy link
Contributor

@c-chaitanya c-chaitanya commented Feb 14, 2021

While going through the documentation on https://plotly.com/python/colorscales/ I realised that this parameter color_continous_midpoint does not exist. This was a typo for color_continuous_midpoint.
While fixing this I noticed a few other misspelt words. So I made a little NLP project for myself and wrote a script which would print all misspelt words to a file. I then went through the spellings and corrected them manually.

@nicolaskruchten
Copy link
Contributor

Thanks very much for these fixes! I'll retarget your PR to the doc-prod branch and merge now.

@nicolaskruchten nicolaskruchten changed the base branch from master to doc-prod February 16, 2021 02:22
@nicolaskruchten nicolaskruchten changed the base branch from doc-prod to master February 16, 2021 02:23
@nicolaskruchten
Copy link
Contributor

Hmm, actually I can't grab just your commit into doc-prod right now, as it's based on master which has unreleased changes. If you could cherry-pick this commit onto doc-prod and submit a new PR I would merge it immediately :)

@c-chaitanya c-chaitanya changed the base branch from master to doc-prod February 16, 2021 03:29
@c-chaitanya c-chaitanya changed the base branch from doc-prod to master February 16, 2021 03:35
@c-chaitanya c-chaitanya changed the base branch from master to doc-prod February 16, 2021 03:37
@c-chaitanya
Copy link
Contributor Author

Hi thanks for the response @nicolaskruchten. Here is the new pull request #3073

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