Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

add templates #42

Merged
merged 1 commit into from
Feb 7, 2022
Merged

Conversation

alanssitis
Copy link
Contributor

What is a quick description of the change?

Added issues template

Is this fixing an issue?

No

Were any issues created as a result of this change?

No

Are there more details that are relevant?

No

Check lists (check x in [ ] of list items)

  • Test written/updating
  • Tests passing
  • Coding style (indentation, etc)

Any additional comments?

Copy link
Member

@zghera zghera left a comment

Choose a reason for hiding this comment

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

Could you add to the checklist in the pull request template? I started a draft PR with some checklist additions in #30 but I wasn't sure what all to include as some things may be verified by CI here soon (so not sure if we should have a checklist for some of them). Give it a look and add what you think makes sense.

Otherwise looks good!

@alanssitis
Copy link
Contributor Author

Changed only one bullet list. The other two should be fixed by Issue #32.

@zghera zghera self-requested a review February 7, 2022 15:19
@alanssitis alanssitis merged commit e23b8fd into Autonomous-Motorsports-Purdue:master Feb 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants