Skip to content

fpr5 - #8

Open
rfay wants to merge 32 commits into
ddev-test:mainfrom
rfay:fpr5
Open

fpr5#8
rfay wants to merge 32 commits into
ddev-test:mainfrom
rfay:fpr5

Conversation

@rfay

@rfay rfay commented Sep 19, 2025

Copy link
Copy Markdown
Member

…ines

Replace CLAUDE.md with symlink to AGENTS.md. Added communication style guidelines including banned vague superlatives and team development patterns from main DDEV repo.

🤖 Developed with assistance from Claude Code

Co-Authored-By: Claude noreply@anthropic.com

rfay and others added 30 commits September 19, 2025 11:11
- Add content validation checks for executable files and unsafe patterns
- Integrate textlint and prettier quality checks before build
- Validate blog post frontmatter structure (title, pubDate, author)
- Add image size warnings for files >2MB
- Fix YAML syntax issues with placeholder HTML generation
- Create comprehensive setup guide with troubleshooting section
- Document required Cloudflare secrets and configuration steps

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
…oject

- Change CF_ACCOUNT_ID and CF_PAGES_PROJECT from secrets to vars references
- Update documentation to clarify secrets vs variables configuration
- This resolves deployment failures due to missing secret references

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Configure workflow to use existing ddev-com-front-end Cloudflare project
- Provides consistent preview URLs with main repository
- Update documentation to reflect unified project approach
- Preview URLs will be: https://pr-{number}.ddev-com-front-end.pages.dev
- Eliminates need for separate Cloudflare project setup

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Update to use TESTS_SERVICE_ACCOUNT_TOKEN (not PUSH_SERVICE_ACCOUNT_TOKEN)
- Clarify CF_API_TOKEN comes from test-secrets vault
- Match actual workflow configuration

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Prefer steps.pages.outputs.url (branch-based) over deployment-url (commit-based)
- Add logging to show which URL type is being used
- Ensure fork PRs get stable pr-{number}.ddev-com-front-end.pages.dev URLs
- Update comment text to indicate URL type for clarity

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Fix comment detection pattern to match actual text 'Cloudflare Pages'
- Add comprehensive logging to debug Cloudflare URL outputs
- Improve comment format with clear PR number and instructions
- Add debugging info to understand why multiple deployments occur

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Fix multiline template string that was breaking YAML parser
- Split long logging lines to meet prettier requirements
- Auto-format file with prettier to ensure consistent style
- Resolve syntax error that was preventing CI from passing

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
…ines

Replace CLAUDE.md with symlink to AGENTS.md. Added communication style guidelines including banned vague superlatives and team development patterns from main DDEV repo.

🤖 Developed with assistance from [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
…ines

Replace CLAUDE.md with symlink to AGENTS.md. Added communication style guidelines including banned vague superlatives and team development patterns from main DDEV repo.

🤖 Developed with assistance from [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 19, 2025

Copy link
Copy Markdown

🌐 Fork Preview for PR #8

https://e9257135.ddev-com-fork-previews.pages.dev

This preview updates automatically when you push changes to your fork.

@rfay
rfay force-pushed the main branch 2 times, most recently from 0ce7fe5 to a118f0b Compare September 22, 2025 13:12
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.

1 participant