10up / component-library

A library of barebones front-end components built with WordPress and accessibility in mind.
https://baseline.10up.com/
MIT License
59 stars 12 forks source link

Update workflow documentation in contributing page #74

Open joesnellpdx opened 3 years ago

joesnellpdx commented 3 years ago

Update workflow documentation on contributing page.

If this is too verbose / complicated - happy to trim per that guidance

Closes #74

joesnellpdx commented 3 years ago

@nicholasio

Added you to review this update. I may have gone a bit overboard. Happy to alter in any way you see fit.

github-actions[bot] commented 3 years ago

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 0 0
Ignored 0 N/A

Report generated by eslint-plus-action

cypress[bot] commented 3 years ago



Test summary

9 0 1 0


Run details

Project @10up/component-library
Status Passed
Commit 773d7b4715 ℹ️
Started Jun 10, 2021 9:18 PM
Ended Jun 10, 2021 9:18 PM
Duration 00:37 💡
OS Linux Ubuntu - 20.04
Browser Chrome 91

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

codecov-commenter commented 3 years ago

Codecov Report

Merging #74 (f345c2a) into develop (27199b3) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #74   +/-   ##
========================================
  Coverage    66.82%   66.82%           
========================================
  Files           12       12           
  Lines          853      853           
  Branches       196      196           
========================================
  Hits           570      570           
  Misses         229      229           
  Partials        54       54           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 97f3409...f345c2a. Read the comment docs.

joesnellpdx commented 3 years ago

@nicholasio I've updated the documentation... I left some of your comments open for quick reference to the prior dialog. Please review when you have a moment. Hopefully, I have better captured your intention / workflow.