-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
gh-131170: fix duplicated sections in frames.md file #131177
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
Conversation
In the frames.md file at InternalDocs, change the first duplicated title to Specials, and remove unnecessary content under the Specials section.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me.
@@ -0,0 +1,2 @@ | |||
The duplication of titles in the frames.md file at InternalDocs has been |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for a NEWS entry as it's internal documentation and not exposed documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you. However, it would be a good idea to add this comment as a note in the devguide, specifically in the quick reference at step 8. 👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean, the fact that it's an internal-only?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess it doesn't hurt to specify when a NEWS entry is really needed. Thanks for the feedback!
3200efd
to
99d7808
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
I was planning to merge it after eating something but you beat me to it! |
In fact, I was eating while merging 🥘 |
…1177) fix duplicated sections in frames.md file In the frames.md file at InternalDocs, change the first duplicated title to Specials, and remove unnecessary content under the Specials section.
…1177) fix duplicated sections in frames.md file In the frames.md file at InternalDocs, change the first duplicated title to Specials, and remove unnecessary content under the Specials section.
In the "frames.md" file at InternalDocs, I have changed the first duplicated title to Specials and removed the unnecessary content under the Specials section.
InternalDocs/frames.md
#131170