Skip to content

Conversation

@f-hollow
Copy link
Collaborator

@f-hollow f-hollow commented Apr 16, 2025

Description

This PR updates the following:

  • When creating an article using the hugo new content command, it won't add a timezone and will default to midnight (UTC+0).
  • In the contribution guide instructions, added YYYY/MM to hugo new content example.
  • Implemented CI checks:
    • Articles added in content/blog/ need to be in the current content/blog/YYYY/MM/ folder
    • Articles need to bear today's date without a timezone

Note

The commit test: add articles with issues in dates is for demo purposes only. It will be removed before merging.

Related

Testing

The workflow has been thoroughly tested in this PR.


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@f-hollow f-hollow force-pushed the ci/add_articles_in_yyyy/mm_folder branch 14 times, most recently from 89165b5 to a90c25c Compare April 16, 2025 07:26
@f-hollow f-hollow requested review from FBEZ and pedrominatel April 16, 2025 07:31
@pedrominatel pedrominatel requested a review from Copilot April 16, 2025 07:35
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

@f-hollow f-hollow force-pushed the ci/add_articles_in_yyyy/mm_folder branch from a90c25c to b124b75 Compare April 16, 2025 10:12
Copy link
Member

@pedrominatel pedrominatel left a comment

Choose a reason for hiding this comment

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

Thanks @f-hollow for preparing this enhancement! LGTM.

@f-hollow f-hollow force-pushed the ci/add_articles_in_yyyy/mm_folder branch from b124b75 to cf3e626 Compare April 17, 2025 02:52
@FBEZ
Copy link
Collaborator

FBEZ commented Apr 17, 2025

Nice, it works also on my machines.
I added a minor suggestion about adding the summary to the front-matter by default.

@f-hollow
Copy link
Collaborator Author

I set the check to make sure that the article date is today. Just because we already started merging articles with future dates, I will change the article date check to today or any time in the future.

@f-hollow f-hollow force-pushed the ci/add_articles_in_yyyy/mm_folder branch 2 times, most recently from 353aa2b to 3ae6e1a Compare April 17, 2025 11:43
@f-hollow
Copy link
Collaborator Author

As long as you agree on the summary, this PR is ready for merge.

@f-hollow f-hollow force-pushed the ci/add_articles_in_yyyy/mm_folder branch 2 times, most recently from a8e8f41 to a1bc89f Compare April 18, 2025 08:05
@f-hollow f-hollow merged commit e834efa into espressif:main Apr 18, 2025
6 checks passed
@f-hollow f-hollow deleted the ci/add_articles_in_yyyy/mm_folder branch April 18, 2025 08:09
@f-hollow f-hollow mentioned this pull request May 30, 2025
6 tasks
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