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

repo: mandatory issue templates (AIDM-427) #3003

Merged
merged 3 commits into from
Jan 21, 2025

Conversation

tlhunter
Copy link
Member

Description

  • makes using issue templates mandatory
  • adds a security document
  • standardizes the issue creation experience across all tracers
  • see the node.js create issue screen for an example of what this looks like

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@tlhunter tlhunter requested a review from a team as a code owner December 18, 2024 00:02
@codecov-commenter
Copy link

codecov-commenter commented Dec 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.78%. Comparing base (6b3218e) to head (b8d19d9).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3003      +/-   ##
============================================
- Coverage     74.80%   74.78%   -0.02%     
  Complexity     2784     2784              
============================================
  Files           112      112              
  Lines         11020    11020              
============================================
- Hits           8243     8241       -2     
- Misses         2777     2779       +2     
Flag Coverage Δ
tracer-php 74.78% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b3218e...b8d19d9. Read the comment docs.

@tlhunter tlhunter force-pushed the tlhunter/create-issue-standard branch from 0ca7d3b to 3748ce5 Compare January 14, 2025 00:29
@tlhunter tlhunter force-pushed the tlhunter/create-issue-standard branch from 3748ce5 to b8d19d9 Compare January 17, 2025 18:40
Copy link
Contributor

@PROFeNoM PROFeNoM left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@bwoebi bwoebi left a comment

Choose a reason for hiding this comment

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

This is okay.

@tlhunter tlhunter merged commit 6b36477 into master Jan 21, 2025
648 of 689 checks passed
@tlhunter tlhunter deleted the tlhunter/create-issue-standard branch January 21, 2025 19:53
@github-actions github-actions bot added this to the 1.7.0 milestone Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants