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.

Sign up for GitHub

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

Jump to bottom

New , form-based Issue templates for Github repository #6919

Merged
merged 1 commit into from
Oct 2, 2023
Merged

New , form-based Issue templates for Github repository #6919

merged 1 commit into from
Oct 2, 2023

Conversation

pktiuk
Copy link
Contributor

@pktiuk pktiuk commented Sep 28, 2023 •
edited
Loading

Motivation and context

I think a proper issue template based on simple form would make creating a proper issues easier for users.

How has this been tested?

These templates can be checked on my fork of this repo: https://github.com/pktiuk/cvat/issues/new/choose

image

Checklist

  • I submit my changes into the develop branch
  • I have added a description of my changes into the CHANGELOG file
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)
  • I have increased versions of npm packages if it is necessary
    (cvat-canvas,
    cvat-core,
    cvat-data and
    cvat-ui)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@pktiuk pktiuk requested a review from mdacoca as a code owner September 28, 2023 09:13
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

Merging #6919 (de25c85) into develop (4a487c3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #6919   +/-   ##
========================================
  Coverage    82.52%   82.52%           
========================================
  Files          370      370           
  Lines        39869    39869           
  Branches      3560     3560           
========================================
  Hits         32902    32902           
  Misses        6967     6967
Components Coverage Δ
cvat-ui 77.56% <ø> (ø)
cvat-server 86.88% <ø> (ø)

nmanovic
nmanovic approved these changes Oct 2, 2023
View reviewed changes
Copy link
Contributor

@nmanovic nmanovic left a comment

Choose a reason for hiding this comment

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

@pktiuk , thank you. It looks great! 🥇

@nmanovic nmanovic merged commit 78d45fb into cvat-ai:develop Oct 2, 2023
@pktiuk
Copy link
Contributor Author

pktiuk commented Oct 2, 2023

@pktiuk , thank you. It looks great! 🥇

I'm glad, you like it :)

@pktiuk pktiuk deleted the issue_template branch October 2, 2023 07:27
mikhail-treskin pushed a commit to retailnext/cvat that referenced this pull request Oct 25, 2023
@pktiuk @mikhail-treskin
New , form-based Issue templates for Github repository (cvat-ai#6919)
49b6fd0 
I think a proper issue template based on simple form would make creating
a proper issues easier for users.

These templates can be checked on my fork of this repo:
https://github.com/pktiuk/cvat/issues/new/choose

![image](https://github.com/opencv/cvat/assets/45544416/ffe2a64c-1cc6-4834-8ec9-9548144a21f5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nmanovic nmanovic nmanovic approved these changes

@sizov-kirill sizov-kirill Awaiting requested review from sizov-kirill

@mdacoca mdacoca Awaiting requested review from mdacoca

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pktiuk @nmanovic