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

Added Labels Dynamically,added comments explaining what each pattern … #6149

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Zaidco
Copy link

@Zaidco Zaidco commented Jan 21, 2025

…detects,corrected typos,added place holder to guid users for a better experiece

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java

…detects,corrected typos,added place holder to guid users for a better experiece
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.84%. Comparing base (364f660) to head (98c5be9).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #6149   +/-   ##
=========================================
  Coverage     73.83%   73.84%           
- Complexity     5155     5156    +1     
=========================================
  Files           661      661           
  Lines         17687    17687           
  Branches       3409     3409           
=========================================
+ Hits          13060    13061    +1     
  Misses         4119     4119           
+ Partials        508      507    -1     

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

@siriak
Copy link
Member

siriak commented Jan 28, 2025

What did you do? There's a field called context already present in the template

attributes:
label: "Additional context"
description: "Is there anything else we should know about this bug report?"
placeholder: "E.g., recent changes in your setup, related logs, etc."
Copy link

@anandfresh anandfresh Jan 30, 2025

Choose a reason for hiding this comment

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

Just a suggestion to update description text
Do you have any further information on this bug report?

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