Skip to content
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

Chore/issue 17 go live checklist #26

Merged
merged 9 commits into from
May 13, 2022

Conversation

thescientist13
Copy link
Member

@thescientist13 thescientist13 commented May 12, 2022

Related Issue

resolves #17

Summary of Changes

  1. Setup project for npm publishing (wcc was already taken 😢 )
  2. Clean up TODOs + commented code
  3. PrismJS for code samples
  4. Pull simple.css from npm / node modules
  5. tags and Lighthouse audit (note: PrismJS theme has known contrast issues - Theme accessibility PrismJS/prism#3180)
  6. Clean up comments and TODO

TODO

  1. Contributing Guide
  2. Test contrast changes at night for dark mode compatibility

@thescientist13 thescientist13 mentioned this pull request May 12, 2022
15 tasks
@thescientist13 thescientist13 linked an issue May 12, 2022 that may be closed by this pull request
15 tasks
@thescientist13 thescientist13 merged commit ad5730b into master May 13, 2022
@thescientist13 thescientist13 deleted the chore/issue-17-go-live-checklist branch May 13, 2022 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Go Live checklist
1 participant