Skip to content

Conversation

@hi-ogawa
Copy link
Contributor

Add comprehensive contributing guide for @vitejs/plugin-rsc covering:

  • New e2e testing approach using setupInlineFixture
  • Development workflow and testing patterns
  • Common patterns for server actions, HMR, CSS testing
  • Tips for contributors and coding agents

Also add link from root CONTRIBUTING.md to plugin-rsc guide.

🤖 Generated with Claude Code

Description

Add comprehensive contributing guide for @vitejs/plugin-rsc covering:
- New e2e testing approach using setupInlineFixture
- Development workflow and testing patterns
- Common patterns for server actions, HMR, CSS testing
- Tips for contributors and coding agents

Also add link from root CONTRIBUTING.md to plugin-rsc guide.

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

Co-Authored-By: Claude <[email protected]>
@hi-ogawa hi-ogawa changed the title docs(plugin-rsc): add contributing guide with dev tips docs(rsc): add CONTRIBUTING.md Jul 23, 2025
hi-ogawa and others added 5 commits July 23, 2025 11:39
- Remove Project Structure section and verbose code examples
- Reference existing test files instead of inline examples
- Fix test commands to use correct test-e2e script name
- Add --ui and -g filter options for better developer experience
- Streamline content for contributors and coding agents

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

Co-Authored-By: Claude <[email protected]>
- Put examples/basic first as it's "mainly used for e2e testing" per README
- Position setupInlineFixture as option for specific edge cases
- Better align guide with actual project structure and usage patterns

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

Co-Authored-By: Claude <[email protected]>
@hi-ogawa hi-ogawa marked this pull request as ready for review July 23, 2025 03:03
@hi-ogawa hi-ogawa merged commit 4005dbe into main Jul 23, 2025
18 checks passed
@hi-ogawa hi-ogawa deleted the add-rsc-contributing-guide branch July 23, 2025 03:18
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