Skip to content

Conversation

@dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Feb 9, 2026

  • ci: fix content type for pagefind fragments

@netlify
Copy link

netlify bot commented Feb 9, 2026

👷 Deploy Preview for docsdocker processing.

Name Link
🔨 Latest commit 262ad2d
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/698a08b790faa70008bf729f

@dvdksn dvdksn force-pushed the fix-pagefind-fragments-contenttype branch from 262ad2d to 37f5011 Compare February 9, 2026 16:18
@github-actions github-actions bot added area/release Relates to CI or deployment ci ci updates labels Feb 9, 2026
@netlify
Copy link

netlify bot commented Feb 9, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 262ad2d
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/698a08b790faa70008bf729f
😎 Deploy Preview https://deploy-preview-24104--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Feb 9, 2026

👷 Deploy Preview for docsdocker processing.

Name Link
🔨 Latest commit 37f5011
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/698a08cb7c8fe5000810ba56

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Review Summary

This PR adds CI/CD infrastructure for handling pagefind search index files with proper compression headers. The changes are technically sound.

The workflow correctly:

  • Excludes pagefind index files from the main S3 sync
  • Uploads them separately with gzip encoding and appropriate content-type
  • Uses correct AWS CLI syntax (filter parameters can appear after paths)

No documentation issues found in the changed content.

@netlify
Copy link

netlify bot commented Feb 9, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 37f5011
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/698a08cb7c8fe5000810ba56
😎 Deploy Preview https://deploy-preview-24104--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Member

@usha-mandya usha-mandya left a comment

Choose a reason for hiding this comment

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

LGTM

@dvdksn dvdksn merged commit d596044 into docker:main Feb 9, 2026
14 of 15 checks passed
@dvdksn dvdksn deleted the fix-pagefind-fragments-contenttype branch February 9, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/release Relates to CI or deployment ci ci updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants