-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Update blog content files to move author details in front-matter #45957
base: main
Are you sure you want to change the base?
Update blog content files to move author details in front-matter #45957
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
/lgtm
Since this will standardize the format, Thanks @dipesh-rawat
LGTM label has been added. Git tree hash: 20556819bfb983dcb45865434dd5dd1c29f64e0f
|
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.
To make this easier to review, I recommend splitting it by the year of the article.
Would you do that @dipesh-rawat?
I agree with @sftim |
903c475
to
36ac34a
Compare
New changes are detected. LGTM label has been removed. |
Sure, I've done that now. |
36ac34a
to
e3f60b5
Compare
Reviewing over 350+ files will be a substantial task for this PR. To tackle this efficiently, I'll be breaking down the pull request into individual ones (each focusing on a single year), starting from this year’s blogs and progressing in descending order. |
e3f60b5
to
0c9c296
Compare
9a72642
to
379fa64
Compare
379fa64
to
d630747
Compare
d630747
to
7a2793b
Compare
7a2793b
to
c8374f6
Compare
c8374f6
to
3993912
Compare
3993912
to
f9d9ebf
Compare
This PR extends the changes made in PR #45865 and implements those changes to all the blog files as per the recommendation in #45865 (comment). The primary change involves relocating author details into the metadata within the front matter.
Edit: I'll split this into several PRs each focusing on a single year blog.
Useful Link
Preview Blog Summary Page