feat(actions): added semantic PR check workflow. Fixes #47

rohankmr414 commented 2 years ago

Signed-off-by: Rohan Kumar rohankmr414@gmail.com

Fixes #47

rohankmr414 commented 2 years ago

@halcyondude can you disable the approval requirement for running action workflows for outside forks?

rohankmr414 commented 2 years ago

still requires approval

halcyondude commented 2 years ago

hrm...just re-saved this...

halcyondude commented 2 years ago

at this point AFAIK you shouldn't need another approval, just the first time.

halcyondude commented 2 years ago

This is pretty cool! TIL :)

<type>[optional scope]: <description>

[optional body]

[optional footer(s)]

Available types:

feat: A new feature
fix: A bug fix
docs: Documentation only changes
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
refactor: A code change that neither fixes a bug nor adds a feature
perf: A code change that improves performance
test: Adding missing tests or correcting existing tests
build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
chore: Other changes that don't modify src or test files
revert: Reverts a previous commit

rohankmr414 commented 2 years ago

looks ready to merge I guess?

halcyondude commented 2 years ago

@rohankmr414 afaik after this merges, new PR's from you shouldn't cause more approvals.

Thanks for your contribution! This one makes all manner of sense :)