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

Update template deps, Dockerfile and workflow #4

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mhanslo-paystack
Copy link

Pull request template

Please follow the template instructions. But also take the time to delete the irrelevant parts from this template so that there's no clutter.

Describe the PR

  • A clear and concise summary of what this PR is about and the technical approach it takes
  • Link to the issue that requires this PR
  • Explicitly explain if this is a backwards incompatible or breaking change
  • Add extra information and screenshots if you believe it adds value
  • Mark this as a Draft PR if you just want to get early feedback on your approach
  • Use the do not merge label if this is no longer a draft, you want approvals, but don't want it to be merged immediately, remove the label once that's done
  • If any of the sign off items are unchecked, please draw special attention to it!

Sign off

  • Code conforms to the Frontend Coding Style Guide
  • Added test(s)
  • Merged this branch into dev
  • Added sufficient inline comments and/or added to DEV_DOCS.md if needed
  • Can be safely merged to master and deployed to production pending pr approval

@mhanslo-paystack mhanslo-paystack changed the title Chore/update template Update template deps, Dockerfile and workflow Nov 1, 2023
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.

1 participant