Skip to content

fix: include source in dockerfile - #14

Merged
mezotv merged 2 commits into
mainfrom
fix/docker-source-build
Jul 18, 2025
Merged

fix: include source in dockerfile#14
mezotv merged 2 commits into
mainfrom
fix/docker-source-build

Conversation

@mezotv

@mezotv mezotv commented Jul 18, 2025

Copy link
Copy Markdown
Member

Includes the source and mdx config in the docker build and switches back to an img component with explanation .

@mezotv
mezotv requested a review from tobias-hay as a code owner July 18, 2025 06:41
@mezotv
mezotv requested a review from Copilot July 18, 2025 06:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 fixes Docker build issues by including necessary source configuration files and resolves a compatibility issue with the next/og library by switching from Next.js Image component to a standard img element.

  • Includes source.config.ts and mdx-components.tsx files in the Docker build process
  • Replaces next/image Image component with standard <img> element in OpenGraph generation
  • Adds proper linting ignore comment with explanation for the img element usage

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
Dockerfile Adds copying of source configuration and MDX component files to fix build dependencies
app/og/[...slug]/og.tsx Switches from Next.js Image to standard img element for OpenGraph compatibility

Comment thread app/og/[...slug]/og.tsx Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mezotv
mezotv merged commit 32d62aa into main Jul 18, 2025
0 of 2 checks passed
@mezotv
mezotv deleted the fix/docker-source-build branch July 18, 2025 06:42
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