Skip to content

feat: Add basic blog functionality#1

Open
FebilAravindan wants to merge 1 commit into
masterfrom
add-blog-feature
Open

feat: Add basic blog functionality#1
FebilAravindan wants to merge 1 commit into
masterfrom
add-blog-feature

Conversation

@FebilAravindan

Copy link
Copy Markdown
Owner
  • Created a 'blog' directory for markdown posts.
  • Added a sample blog post 'my-first-post.md'.
  • Created an HTML template 'blog-post-template.html' for styling posts.
  • Created 'blog.html' to list blog posts.
  • Added a Python script 'convert_md_to_html.py' to convert markdown posts to HTML using the template.
  • Installed 'markdown' library.
  • Generated 'blog/my-first-post.html' from the markdown file.
  • Updated 'index.html' with a navigation link to the blog page.
  • Verified all links and content display as expected.

- Created a 'blog' directory for markdown posts.
- Added a sample blog post 'my-first-post.md'.
- Created an HTML template 'blog-post-template.html' for styling posts.
- Created 'blog.html' to list blog posts.
- Added a Python script 'convert_md_to_html.py' to convert markdown posts to HTML using the template.
- Installed 'markdown' library.
- Generated 'blog/my-first-post.html' from the markdown file.
- Updated 'index.html' with a navigation link to the blog page.
- Verified all links and content display as expected.
@FebilAravindan FebilAravindan requested a review from Copilot June 28, 2025 16:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds basic blog functionality by integrating markdown-based posts with HTML templates and updating navigation.

  • Adds a blog directory with sample markdown and HTML files.
  • Implements a Python script to convert markdown files into styled HTML using a new template.
  • Introduces a new blog listing page and updates main navigation accordingly.

Reviewed Changes

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

Show a summary per file
File Description
index.html Updates navigation to include a link to the blog page.
convert_md_to_html.py Introduces markdown-to-HTML conversion with metadata handling.
blog/my-first-post.md Provides a sample markdown blog post with metadata.
blog/my-first-post.html Generated HTML page from the markdown post.
blog.html Creates a blog listing page for posts.
blog-post-template.html Adds an HTML template for styling individual blog posts.

Comment thread index.html
</div>
<div class="nav_link">
blog
<a href="blog.html" style="color: inherit; text-decoration: none;">blog</a>

Copilot AI Jun 28, 2025

Copy link

Choose a reason for hiding this comment

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

Consider moving inline styles to a CSS class to improve maintainability and consistency across the project.

Copilot uses AI. Check for mistakes.
Comment thread convert_md_to_html.py
Comment on lines +67 to +68
if not os.path.exists(output_dir):
os.makedirs(output_dir)

Copilot AI Jun 28, 2025

Copy link

Choose a reason for hiding this comment

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

Consider using os.makedirs(output_dir, exist_ok=True) to simplify directory creation and handle concurrent directory creation safely.

Suggested change
if not os.path.exists(output_dir):
os.makedirs(output_dir)
os.makedirs(output_dir, exist_ok=True)
if not os.path.exists(output_dir): # Log only if the directory was just created

Copilot uses AI. Check for mistakes.
Comment thread blog/my-first-post.html
<div class="container">
<p class="post-meta">Published on: 2023-10-27 by Your Name</p>
<div class="post-content">
<h1>My First Post</h1>

Copilot AI Jun 28, 2025

Copy link

Choose a reason for hiding this comment

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

[nitpick] The title appears both in the header and within the post content, which might lead to duplication; consider removing one instance to streamline the presentation.

Suggested change
<h1>My First Post</h1>

Copilot uses AI. Check for mistakes.
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.

2 participants