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

Add issue templates for bugs and features #166

Merged
merged 1 commit into from
Aug 21, 2023

Conversation

urrsk
Copy link
Member

@urrsk urrsk commented Aug 15, 2023

No description provided.

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (17a8aae) 70.61% compared to head (c608cb4) 70.61%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #166   +/-   ##
=======================================
  Coverage   70.61%   70.61%           
=======================================
  Files          69       69           
  Lines        2525     2525           
  Branches      322      322           
=======================================
  Hits         1783     1783           
  Misses        556      556           
  Partials      186      186           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@RobertWilbrandt RobertWilbrandt left a comment

Choose a reason for hiding this comment

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

This all seems reasonable to me and i could not think of something important missing. I marked some small parts that stuck out to me, but i am happy with merging this.

- type: textarea
id: description
attributes:
label: Feature summery
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
label: Feature summery
label: Feature summary

- type: dropdown
id: robot-platform
attributes:
label: What robot platform are the library connected to.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
label: What robot platform are the library connected to.
label: Which robot platforms is the library connected to.

label: Issue details
description: This part is important in order for us to understand the problem and to clarify the severity of the issue.
value: |
# Summery
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# Summery
# Summary

Thanks for taking the time to fill out this bug report. Before submitting the bug, please first search on the [issue track](../) before creating one.

Please fill out the following as much as possible, to help us reproducing the problem.
Alternatively, if you do not want to share the issue public send a email to ROS@universal-robots.com with the same information.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Alternatively, if you do not want to share the issue public send a email to ROS@universal-robots.com with the same information.
Alternatively, if you do not want to share the issue publicly, send a email to ROS@universal-robots.com with the same information.

@urrsk urrsk merged commit 22a4fb1 into UniversalRobots:master Aug 21, 2023
17 checks passed
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.

None yet

2 participants