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

Issue forms, pull request template. #43

Merged
merged 26 commits into from
Mar 8, 2023
Merged

Issue forms, pull request template. #43

merged 26 commits into from
Mar 8, 2023

Conversation

Tech-TTGames
Copy link
Owner

Description

I have migrated the normal .md issue forms to the GitHub .yaml issue forms to ensure ease of access for anyone using the reporting tools.

Motivation and Context

I spent my time doing this becouse I forgot to commit the /dev branch changes from my F drive. So I was stuck with no access to the updated code.

Changes Made

  • Migrated .md issue templates to .yaml issue forms
  • Separated Enhancement and Feature templates/forms.
  • Added pull_request_template.md
  • Added a question issue form.

Issues Resolved

  • N/A

Testing Instructions

  • N/A (No changes in code)

Checklist

  • [N/A] I have reviewed my own code and ensured it passes all linting tests.
  • [N/A] I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • [N/A] I have added tests that prove my fix is effective or that my feature works.
  • I have run all tests and ensured they pass.

Currently, r/google if the bot grows, maybe somewhere else.

Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
Signed-off-by: Tech. TTGames <[email protected]>
@Tech-TTGames Tech-TTGames added Type: Documentation Improvements or additions to documentation. Status: Ready The fix/feature has been created and awaits deployment. labels Mar 8, 2023
@Tech-TTGames Tech-TTGames requested a review from kk5dire March 8, 2023 14:04
@Tech-TTGames Tech-TTGames self-assigned this Mar 8, 2023
@Tech-TTGames Tech-TTGames enabled auto-merge March 8, 2023 14:07
@Tech-TTGames Tech-TTGames added the Priority: 3 Issue does not require any attention and will be handled in free time. label Mar 8, 2023
kk5dire
kk5dire previously requested changes Mar 8, 2023
.github/ISSUE_TEMPLATE/config.yml Outdated Show resolved Hide resolved
Copy link
Collaborator

@benfoster04 benfoster04 left a comment

Choose a reason for hiding this comment

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

Please see conversation regarding config.yml

.github/ISSUE_TEMPLATE/config.yml Outdated Show resolved Hide resolved
@kk5dire
Copy link
Collaborator

kk5dire commented Mar 8, 2023

Will approve merge when @benfoster04's CR is resolved.

@Tech-TTGames Tech-TTGames disabled auto-merge March 8, 2023 16:26
invite link removed till a separate server for T+ support is established
@Tech-TTGames Tech-TTGames requested a review from benfoster04 March 8, 2023 16:28
@Tech-TTGames Tech-TTGames dismissed stale reviews from benfoster04 and kk5dire March 8, 2023 16:30

Resolved

@Tech-TTGames Tech-TTGames enabled auto-merge March 8, 2023 16:31
Copy link
Collaborator

@benfoster04 benfoster04 left a comment

Choose a reason for hiding this comment

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

LGTM.
We will organise formal support contact links in due course.
Cleared for merge

@Tech-TTGames Tech-TTGames merged commit 9514b0d into main Mar 8, 2023
@Tech-TTGames Tech-TTGames deleted the issues/templates branch March 8, 2023 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 3 Issue does not require any attention and will be handled in free time. Status: Ready The fix/feature has been created and awaits deployment. Type: Documentation Improvements or additions to documentation.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants