-
Notifications
You must be signed in to change notification settings - Fork 293
Update website documentation to reflect current Echo v4 state #372
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
vishr
wants to merge
13
commits into
master
Choose a base branch
from
update-documentation
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Updated Go version requirement from 1.13 to 1.20 in quick-start guide - Enhanced Hello World example with proper middleware setup - Improved introduction to focus on examples repository purpose - Added comprehensive example categorization and better resource links - Enhanced graceful shutdown documentation with implementation details - Improved JWT authentication cookbook with clearer feature descriptions - Added CLAUDE.md file for future development guidance 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Remove executable permissions from SVG image files as they should be non-executable. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
## New Examples Added: - **Testing cookbook**: Complete testing guide with unit, integration, and validation tests - **Performance cookbook**: Benchmarking, profiling, and load testing examples ## Modernization Updates: - Fixed hardcoded Echo/3.0 version to Echo/4.11 in middleware example - Added environment variable support for JWT secrets in authentication examples ## Test Coverage Added: - **hello-world**: Basic functionality and benchmark tests - **middleware**: Stats middleware and server header tests - **graceful-shutdown**: Integration tests for shutdown behavior - **jwt/custom-claims**: Comprehensive JWT authentication testing - **testing**: Meta-example showing how to test Echo applications - **performance**: Benchmark and performance validation tests ## Enhanced Development Experience: - Updated Makefile with new commands: test-cookbook, benchmark, test-cover - Added dependency management and cleanup targets - Created individual go.mod files for complex examples ## Test Commands Available: - `make test-cookbook` - Run all cookbook tests - `make benchmark` - Run performance benchmarks - `make test-cover` - Generate coverage reports - `make deps` - Install test dependencies 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Updated GitHub Actions to test Go 1.20 and 1.21 - Added cookbook-specific testing to CI pipeline - Fixed go.mod dependencies for testing and performance examples - Enhanced CI to run benchmarks and comprehensive test suite - Updated GitHub Actions to latest versions (v4) The CI now runs: - Standard tests (make test) - Cookbook tests (make test-cookbook) - Performance benchmarks (make benchmark) - Dependency installation (make deps) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix Go version strings in CI (1.20, 1.21 -> "1.20", "1.21") - Run go mod tidy to fix module dependencies - Resolve CI failure caused by go mod requirements 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Updated minimum Go version from 1.20 to 1.21 (main Echo uses 1.23) - Updated CI to test Go 1.21, 1.22, 1.23 (Echo supports last 4 major releases) - Updated all go.mod files in cookbook examples - Updated documentation to reflect Go 1.21+ requirement - Ensures compatibility with Echo project standards 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Updated go.mod to Go 1.25.1 (latest stable) - Updated CI to test Go 1.22, 1.23, 1.24, 1.25 (last 4 major releases) - Updated all cookbook examples to Go 1.25.1 - Updated documentation to reflect Go 1.22+ requirement - Aligned with Echo project's version support policy This ensures we're using the latest Go features and maintaining compatibility with Echo's supported version range. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Added workflow_dispatch trigger to allow manual CI runs via GitHub CLI. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Replace manual server startup with httptest.Server for reliable testing - Eliminate race conditions when accessing e.Listener.Addr() - Reduce test execution time while maintaining test coverage - Use proper synchronization patterns for concurrent operations These changes ensure tests are race-free and more reliable in CI environments. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove unused imports (context, fmt) - Add required httptest import - Ensure all imports are properly used 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Set JWT_SECRET environment variable consistently for both login and verification - Ensure login function and JWT middleware use the same secret - Fixes CI test failure where token verification failed due to secret mismatch 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove unused encoding/json import from server.go - Fix function comparison issue in server_test.go by using needsWarmup boolean flag - Resolve "invalid operation: tt.handler == cached" error 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add Clear() method to Cache struct - Clear cache at start of TestCacheEffectiveness to ensure fresh state - Prevents test failures when cache is warmed by previous tests 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code