Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Modified index.rst file of Getting Started #26983

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 1 commit into from

Conversation

15IITian
Copy link

@15IITian 15IITian commented Oct 3, 2023

PR summary

In Getting Started/ index.rst file ->

  • There is a line marked in given photo
    2

  • Here It is working as ** sub_sub_section** , thus I applied the markdown character as shown ->

Untitled design

Output will be like ->
Screenshot from 2023-10-03 22-18-40

PR address the issue #26862

PR checklist

@story645
Copy link
Member

story645 commented Oct 3, 2023

Hi, thanks for the contribution! Can you explain why you're turning that line into a paragraph heading?

ETA: for example does it help discoverability, readibility, etc? If this is an attempt at #26862, that PR is about changing level markers to a standard type and not adding new ones & I will clarify over there.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide

We strive to be a welcoming and open project. Please follow our Code of Conduct.

@jklymak jklymak added the hacktoberfest: spam invalid for hacktoberfest label Oct 3, 2023
@jklymak
Copy link
Member

jklymak commented Oct 3, 2023

Thank you for your interest in Matplotlib and contributing to open-source. I've marked this "hacktoberfest: spam" as we are actively trying to discourage trivial contributions like this. Targeting anything in our top level for minor wording or formatting changes will not be accepted.

If you would like to make a substantive contribution, your help would be most welcome. We have a Good First Issue tag. We also ask that you read our contributor guide

If this contribution is in earnest, thanks, but note it will not count towards hacktoberfest.

Thanks for your understanding.

@15IITian
Copy link
Author

15IITian commented Oct 3, 2023

Thank you for your response @story645
Actually when I read it for first time , I thought that it functions as a sub section of Draw a first plot

and If my work is not on right way -> Then please clarify/ guide me .

@story645
Copy link
Member

story645 commented Oct 3, 2023

Actually when I read it for first time , I thought that it functions as a sub section of Draw a first plot

Yes, but is the only content in that section and acts to introduce the code in the example. There's no need for subsections if you have only one section.

If you can make it, we have a new contributors meeting in about 20 minutes (5PM UTC) where you can ask questions. Also please look at the contributor guide Jody linked to, it will have the link to the meeting.

@story645 story645 removed the hacktoberfest: spam invalid for hacktoberfest label Oct 4, 2023
@story645
Copy link
Member

story645 commented Oct 4, 2023

I'm gonna close this because I don't think there will be consensus for the change, but I removed the hacktober:spam label because I believe/hope this was an earnest attempt at starting with a smaller issue to get familiar with our build system. This PR can always be reopened if there's consensus that this improves the documentation.

@story645 story645 closed this Oct 4, 2023
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