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

chore: update pr templates and issues #164

Merged
76 changes: 76 additions & 0 deletions .github/ISSUE_TEMPLATE/01_bug.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,76 @@
name: Bug issue
description: Report a bug affecting cuHacking 2025 (includes website, portal and docs)
title: '[BUG]: description-of-issue'
labels:
- bug
body:
- type: checkboxes
id: search
attributes:
label: Search checkbox
description: Insufficiently described issues will be closed, so please provide as much information as possible
options:
- label: Have you searched for similar [issues](https://github.com/cuhacking/2025/issues)?
- type: textarea
id: description
attributes:
label: Description
description: A brief description of the issue
validations:
required: true
- type: textarea
id: str
attributes:
label: Steps to reproduce
description: A series of steps to reproduce the issue
value: |-
1.
2.
3.
validations:
required: true
- type: textarea
id: result
attributes:
label: Actual result
description: What happened
placeholder: include screenshots if needed
validations:
required: true
- type: textarea
id: expectation
attributes:
label: Expected result
description: What should've happened
placeholder: include screenshots if needed
validations:
required: true
- type: dropdown
id: reproduces
attributes:
label: Reproduces how often
multiple: false
options:
- Easily reproduced
- Intermittent issue
- No steps to reproduce
default: 0
validations:
required: true
- type: checkboxes
id: reproducibility
attributes:
label: Reproducibility
description: Does this issue occur...
options:
- label: in Safari?
- label: in Firefox?
- label: in Chrome?
- label: for mobile devices?
- label: for tablet devices?
- label: for desktop devices?
- type: textarea
id: misc
attributes:
label: Miscellaneous information
description: Any additional information, related issues, extra QA steps, configuration or data that might be necessary to reproduce the issue
Comment on lines +72 to +76
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Add a dedicated environment information section.

Consider adding a specific section for environment details before the miscellaneous information.

+  - type: textarea
+    id: environment
+    attributes:
+      label: Environment Information
+      description: Please provide specific details about your environment
+      value: |-
+        - OS and Version:
+        - Browser and Version:
+        - Device Type:
+        - Screen Resolution (if UI-related):
+    validations:
+      required: true
   - type: textarea
     id: misc
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- type: textarea
id: misc
attributes:
label: Miscellaneous information
description: Any additional information, related issues, extra QA steps, configuration or data that might be necessary to reproduce the issue
- type: textarea
id: environment
attributes:
label: Environment Information
description: Please provide specific details about your environment
value: |-
- OS and Version:
- Browser and Version:
- Device Type:
- Screen Resolution (if UI-related):
validations:
required: true
- type: textarea
id: misc
attributes:
label: Miscellaneous information
description: Any additional information, related issues, extra QA steps, configuration or data that might be necessary to reproduce the issue

68 changes: 68 additions & 0 deletions .github/ISSUE_TEMPLATE/02_adr.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
name: Architecture Decision Record
description: Record a design decision
title: '[ADR]: description'
body:
- type: checkboxes
id: search
attributes:
label: Search checkbox
description: Insufficiently described issues will be closed, so please provide as much information as possible
Comment on lines +8 to +9
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Improve search checkbox description

The current description is vague about what constitutes a sufficient description. Consider providing specific guidance.

-      label: Search checkbox
-      description: Insufficiently described issues will be closed, so please provide as much information as possible
+      label: Pre-submission Checklist
+      description: Before submitting, please ensure your ADR meets these requirements:
+        - Clearly states the architectural problem
+        - Provides concrete technical details
+        - Includes implementation considerations
+        - Documents impact on the system
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
label: Search checkbox
description: Insufficiently described issues will be closed, so please provide as much information as possible
label: Pre-submission Checklist
description: Before submitting, please ensure your ADR meets these requirements:
- Clearly states the architectural problem
- Provides concrete technical details
- Includes implementation considerations
- Documents impact on the system

options:
- label: Have you searched for similar [issues](https://github.com/cuhacking/2025/issues)?
- type: dropdown
id: platforms
attributes:
label: Platforms
multiple: true
options:
- all
- Website
- Docs
- Portal
default: 0
validations:
required: true
- type: textarea
id: problem
attributes:
label: Problem Statement
description: A brief description of what the issue of the design decision is
validations:
required: true
- type: textarea
id: decision
attributes:
label: Decision
description: A brief description of the architectural decision taken
validations:
required: true
- type: textarea
id: comparative_table
attributes:
label: Comparative Table of Technologies
description: |-
Create a table comparing the different technologies that were considered and show pros and cons for specific use cases. You can copy the template for a table here:

```bash
| Use Case | Tech 1 | Tech 2 |
|----------|--------|--------|
| Use Case 1 | | |
| Use Case 2 | | |
| Use Case 3 | | |
```
- type: textarea
id: solution
attributes:
label: Solution Summary
description: A summary of the solution of this decision
placeholder: |-
What is there to do next?
What features need to be implemented?
validations:
required: true

- type: textarea
id: alternatives
attributes:
label: Alternative Options
description: Are there any alternative options that were considered?
Comment on lines +64 to +68
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Enhance Alternative Options section

The Alternative Options section should be required and include specific prompts for thorough documentation of alternatives considered.

   - type: textarea
     id: alternatives
     attributes:
       label: Alternative Options
-      description: Are there any alternative options that were considered?
+      description: List all alternative options that were considered, including:
+        - Brief description of each alternative
+        - Reasons for not choosing them
+        - Trade-offs and constraints
+    validations:
+      required: true
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- type: textarea
id: alternatives
attributes:
label: Alternative Options
description: Are there any alternative options that were considered?
- type: textarea
id: alternatives
attributes:
label: Alternative Options
description: List all alternative options that were considered, including:
- Brief description of each alternative
- Reasons for not choosing them
- Trade-offs and constraints
validations:
required: true

52 changes: 52 additions & 0 deletions .github/ISSUE_TEMPLATE/03_feature_request.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
name: Feature Request
description: Request a new feature
title: 'feat(scope-of-issue): description'
labels:
- enhancement
body:
- type: checkboxes
id: search
attributes:
label: Search checkbox
description: Insufficiently described issues will be closed, so please provide as much information as possible
options:
- label: Have you searched for similar [issues](https://github.com/cuhacking/2025/issues)?
Comment on lines +7 to +13
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Add comprehensive pre-submission checklist.

Consider adding more pre-submission checks to ensure quality submissions:

 options:
   - label: Have you searched for similar [issues](https://github.com/cuhacking/2025/issues)?
+  - label: Have you read our contribution guidelines?
+  - label: Is this feature aligned with the project's goals and scope?
+  - label: Have you considered the impact on existing features and users?
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- type: checkboxes
id: search
attributes:
label: Search checkbox
description: Insufficiently described issues will be closed, so please provide as much information as possible
options:
- label: Have you searched for similar [issues](https://github.com/cuhacking/2025/issues)?
- type: checkboxes
id: search
attributes:
label: Search checkbox
description: Insufficiently described issues will be closed, so please provide as much information as possible
options:
- label: Have you searched for similar [issues](https://github.com/cuhacking/2025/issues)?
- label: Have you read our contribution guidelines?
- label: Is this feature aligned with the project's goals and scope?
- label: Have you considered the impact on existing features and users?

- type: dropdown
id: platforms
attributes:
label: Platforms
multiple: true
options:
- all
- Website
- Docs
- Portal
default: 0
validations:
required: true
- type: textarea
id: problem
attributes:
label: Problem Statement
description: A brief description of what the issue of the feature request is
validations:
required: true
- type: textarea
id: solution
attributes:
label: Solution
description: A brief description of the solution taken
validations:
required: true
Comment on lines +34 to +40
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Improve solution section guidance.

The solution description needs more structure:

-description: A brief description of the solution taken
+description: |
+  Describe your proposed solution in detail:
+  - How would this feature work?
+  - What are the expected inputs and outputs?
+  - How would users interact with this feature?
+  - What are the technical requirements?
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- type: textarea
id: solution
attributes:
label: Solution
description: A brief description of the solution taken
validations:
required: true
- type: textarea
id: solution
attributes:
label: Solution
description: |
Describe your proposed solution in detail:
- How would this feature work?
- What are the expected inputs and outputs?
- How would users interact with this feature?
- What are the technical requirements?
validations:
required: true

- type: textarea
id: alternatives
attributes:
label: Alternatives
description: A brief description of any alternatives considered
validations:
required: true
- type: textarea
id: additional_context
attributes:
label: Additional Context
description: Any additional context or information (screenshots can be placed here)
31 changes: 31 additions & 0 deletions .github/ISSUE_TEMPLATE/04_other.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
name: Other
description: Create an issue that doesn't fit into any other category
title: 'type(scope-of-issue): description'
body:
- type: checkboxes
id: search
attributes:
label: Search checkbox
description: Insufficiently described issues will be closed, so please provide as much information as possible
options:
- label: Have you searched for similar [issues](https://github.com/cuhacking/2025/issues)?
- type: dropdown
id: platforms
attributes:
label: Platforms
multiple: true
options:
- all
- Website
- Docs
- Portal
default: 0
validations:
required: true
Comment on lines +12 to +24
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Refine platforms dropdown for better clarity and coverage.

The current platform options and default selection could be improved.

   - type: dropdown
     id: platforms
     attributes:
       label: Platforms
       multiple: true
       options:
-        - all
         - Website
         - Docs
         - Portal
+        - Infrastructure
+        - Database
+        - API
+        - Mobile App
+        - Other (specify in description)
-      default: 0
     validations:
       required: true

Consider removing the "all" option as it might conflict with multiple selection, and users can simply select all applicable options if needed.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- type: dropdown
id: platforms
attributes:
label: Platforms
multiple: true
options:
- all
- Website
- Docs
- Portal
default: 0
validations:
required: true
- type: dropdown
id: platforms
attributes:
label: Platforms
multiple: true
options:
- Website
- Docs
- Portal
- Infrastructure
- Database
- API
- Mobile App
- Other (specify in description)
validations:
required: true

- type: textarea
id: description
attributes:
label: Description
description: A brief description of what needs to be done
validations:
required: true
30 changes: 0 additions & 30 deletions .github/ISSUE_TEMPLATE/architectural-design-record--adr-.md

This file was deleted.

41 changes: 0 additions & 41 deletions .github/ISSUE_TEMPLATE/bug-report.md

This file was deleted.

24 changes: 0 additions & 24 deletions .github/ISSUE_TEMPLATE/build.md

This file was deleted.

24 changes: 0 additions & 24 deletions .github/ISSUE_TEMPLATE/chore.md

This file was deleted.

5 changes: 5 additions & 0 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
blank_issues_enabled: true
contact_links:
- name: cuHacking Website
url: https://cuhacking.ca/
about: Please find all of our media links here to reach out to us.
24 changes: 0 additions & 24 deletions .github/ISSUE_TEMPLATE/continuous-integration--ci.md

This file was deleted.

Loading
Loading