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

feat(instrumentation/report): add problems to reports #28042

Merged
merged 19 commits into from Mar 20, 2024
Merged

feat(instrumentation/report): add problems to reports #28042

merged 19 commits into from Mar 20, 2024

Conversation

secustor
Copy link
Collaborator

Changes

Adds global and repository specific problem lists to the report.

Context

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

secustor and others added 18 commits December 2, 2023 01:19
@secustor
feat(instrumentation/reporting): add report option
2e697f9
@secustor
move types to types.ts
0b9b5aa
@secustor @HonkingGoose
Apply suggestions from code review
5ef4e05 
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
@secustor
Merge branch 'main' into feat/add-reports
18cd3f7
@secustor
Merge branch 'main' into feat/add-reports
bd200c4
@secustor
implement suggestions
6d91bf9
@secustor
add getReport to allow extractions
83e8e19
@secustor
Merge branch 'main' into feat/add-reports
62e02ce
@secustor
Merge remote-tracking branch 'origin/feat/add-reports' into feat/add-…
58db11f 
…reports
@secustor
feat: add tests and add writeSystemFile
b4e6df5
@secustor
feat: move validation and add test for writeSystemFile
28e7bea
@secustor
chore: move s3 mocks to a separate package
d9c11d3
@secustor
Merge main
d7ae6f5
@secustor
test(instrumentation/reporting): do not load recursively s3 and fs pa…
0896e39 
…ckages
@secustor
feat(instrumentation/report): add problems to reports
12e9e89
@secustor
Merge branch 'main' into feat/add-problems-to-report
a588506 
# Conflicts:
#	lib/instrumentation/reporting.spec.ts
#	lib/instrumentation/reporting.ts
#	lib/instrumentation/types.ts
#	lib/workers/global/index.ts
@secustor
chore: remove unnecessary code and fix tests
3be979d
@secustor
chore: clone before manipulating problems list
4224d3b
@secustor secustor requested a review from viceice March 20, 2024 11:26
@rarkins rarkins enabled auto-merge March 20, 2024 12:40
viceice
viceice reviewed Mar 20, 2024
View reviewed changes
lib/instrumentation/reporting.spec.ts Show resolved Hide resolved
lib/instrumentation/reporting.ts Outdated Show resolved Hide resolved
lib/workers/global/index.spec.ts Show resolved Hide resolved
@secustor secustor requested a review from viceice March 20, 2024 16:18
@rarkins rarkins added this pull request to the merge queue Mar 20, 2024
Merged via the queue into renovatebot:main with commit 370927f Mar 20, 2024
37 checks passed
@secustor secustor deleted the feat/add-problems-to-report branch March 20, 2024 16:53
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@viceice viceice viceice approved these changes

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

Successfully merging this pull request may close these issues.

None yet
3 participants
@secustor @viceice @rarkins