Skip to content

fix: add copyright and license headers to .tsx .js .css .html files#37

Closed
tiswanso wants to merge 1 commit intooutshift-open:mainfrom
tiswanso:fix-add-src-copyright-license-headers
Closed

fix: add copyright and license headers to .tsx .js .css .html files#37
tiswanso wants to merge 1 commit intooutshift-open:mainfrom
tiswanso:fix-add-src-copyright-license-headers

Conversation

@tiswanso
Copy link
Copy Markdown

📝 Description

Many files are missing copyright and license headers. Add them via:

docker run --rm --volume $(pwd):/data fsfe/reuse annotate --copyright-prefix string --copyright "Cisco Systems, Inc. and its affiliates" -l "Apache-2.0" --skip-existing **/*.js **/*.ts **/*.tsx **/*.css **/*.html

🔗 Related Issue

n/a

🧪 Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎨 Style/design update
  • 🔧 Build/CI update
  • ♻️ Refactoring (no functional changes)
  • 🧪 Tests only

🧪 Testing

  • Unit tests pass (yarn test)
  • Storybook builds successfully (yarn storybook)
  • Manual testing completed
  • Cross-browser testing (if applicable)

Test Instructions:

♿ Accessibility

  • Follows WCAG 2.1 AA guidelines
  • Keyboard navigation tested
  • Screen reader tested
  • Color contrast verified
  • Focus indicators present
  • ARIA attributes added where needed

📱 Responsive Design

  • Mobile responsive
  • Tablet responsive
  • Desktop responsive
  • All breakpoints tested

📚 Documentation

  • Storybook story added/updated
  • Component props documented
  • Usage examples provided
  • README updated (if needed)
  • TypeScript types exported

🔄 Breaking Changes

  • This PR introduces breaking changes
  • Migration guide provided (if breaking changes)
  • Version bump required

📋 Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Code is commented where necessary
  • No console.log statements left in code
  • Build passes locally
  • Tests added for new functionality
  • Existing tests still pass
  • No TypeScript errors
  • No accessibility violations
  • PR title follows conventional commit format

📸 Screenshots

Before

After

🔍 Additional Notes


For Maintainers:

  • Ready for review
  • Requires design review
  • Requires accessibility review
  • Ready to merge

Signed-off-by: Tim Swanson <tiswanso@cisco.com>
@tiswanso tiswanso requested a review from a team September 22, 2025 19:40
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