Hygiene improvements: CI, deployment, code quality#1
Merged
Conversation
- Add MIT LICENSE and .nvmrc (Node 22) - Refactor 404.html to use shared style.css (171 → 35 lines) - Add 404-specific styles to style.css - Improve semantic markup: <hr> separator, <section> for counter - Add meta author tag to both pages - Update manifest description to match current copy - Update sitemap lastmod date - Bump Service Worker cache to v3
- Add CHANGELOG.md with project history - Add site.webmanifest to Service Worker cache list - Counter API /health endpoint documented in todos/worker.js - Uptime monitoring guide added to todos/monitoring-setup.md
CI & Quality: - Add GitHub Actions workflow for lint/format/validate - Add ESLint config for JavaScript linting - Add Prettier for consistent code formatting - Add html-validate for HTML validation - Add Stylelint for CSS validation Files: - Add humans.txt with project philosophy - Add package.json with dev dependencies - Fix button type attributes for accessibility - Remove redundant ARIA role on section - Apply Prettier formatting to all files - Bump Service Worker cache to v4
- Add build script that injects git SHA into Service Worker cache name
- Update CI workflow with build and deploy jobs
- Deploy to GitHub Pages on push to main
- Use '1mb-dev' as local placeholder, '1mb-{sha}' in production
Flow: validate → build (inject version) → deploy
- Add workers/counter.js with /health endpoint - Add workers/wrangler.toml configuration - Add wrangler as dev dependency - Add npm scripts: worker:dev, worker:deploy, worker:tail Usage: npm run worker:dev # Local development npm run worker:deploy # Deploy to Cloudflare npm run worker:tail # View live logs Secrets required (set via wrangler secret put): - UPSTASH_REDIS_REST_URL - UPSTASH_REDIS_REST_TOKEN
- Add Development section with npm scripts - Add Counter API commands - Add workers/README.md with setup guide - Update LICENSE copyright year
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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
Changes
Licensing & Metadata
.nvmrcfor Node.js 22Code Quality
meta authortagstype="button"attributes for accessibility<hr>separator,<section>for counterCI Pipeline
Deployment
1mb-{sha}cache nameExtras
Test plan