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 github issues templates #545

Merged
merged 1 commit into from
Dec 29, 2021

Conversation

mathiasvr
Copy link
Contributor

Remove old github issue template and add new ones based on those in the main betaflight repo.

Mostly identical to a PR opened it the Betaflight Configurator repo: betaflight/betaflight-configurator#2665.

@haslinghuis haslinghuis added this to the 3.6.0 milestone Nov 25, 2021
@blckmn
Copy link
Member

blckmn commented Nov 26, 2021

AUTOMERGE: (FAIL)

  • github identifies PR as mergeable -> FAIL
  • assigned to a milestone -> PASS
  • cooling off period lapsed -> PASS
  • commit count less or equal to three -> PASS
  • Don't merge label NOT found -> PASS
  • at least one RN: label found -> FAIL
  • Tested label found -> FAIL
  • assigned to an approver -> FAIL
  • approver count at least three -> FAIL

<!-- Specify the version of Betaflight Blackbox Explorer (as displayed in the app) -->
- Blackbox Explorer version:

<!-- If this bug is related to a specific flight controller and/or configuration please specify it here. Create a diff and post it here in a code block. Put (three backticks) at the start and end of the diff block (instructions on how to do a diff: https://oscarliang.com/use-diff-not-dump-betaflight/) -->
Copy link
Member

@ctzsnooze ctzsnooze Nov 26, 2021

Choose a reason for hiding this comment

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

This line seems relevant to a flight controller or firmware issue. I wonder if here, for logs it might be more useful to ask the user to post the log header, or a link to the problematic log file itself?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes that makes sense, I updated the PR to specify a log instead.

@sonarcloud
Copy link

sonarcloud bot commented Dec 2, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@ctzsnooze ctzsnooze merged commit 72a44db into betaflight:master Dec 29, 2021
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.

4 participants