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

Fran b #287

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Fran b #287

wants to merge 2 commits into from

Conversation

DIPAD-Fran-Bryden
Copy link
Collaborator

Proposed changes

Fixing bug identified

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply

  • Bugfix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation Update (change to naming or other documentation)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR.

  • I have added unit tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have checked that my changes have not broken any other functionality
  • I have linked to any issues this PR fixes

Points for review

Add checkboxes here for any specific aspects of your PR you would like to be checked in peer review. If you don't specify anything here, the reviewer will check:

  • All of your unit tests pass
  • Your code is clear and easy to understand
  • You have documented any new features
  • They can run the code you have written
  • Your changes do not break any other functionality

Who is reviewing this PR?

@LAURMURPH

@DIPAD-Fran-Bryden DIPAD-Fran-Bryden linked an issue Jul 30, 2024 that may be closed by this pull request
@DIPAD-Fran-Bryden DIPAD-Fran-Bryden removed a link to an issue Jul 30, 2024
Copy link

@LAURMURPH LAURMURPH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make the sun go away, I am too hot.

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