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

[SASD-511] fix: json output to console #120

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Torsten1014
Copy link
Collaborator

  • fix: Append filtered projects to an array (results) and then log results after all of the projects have been filtered. Before objects were logged to the console individually rather than as an array resulting in invalid json.

  • fix: Change promise state to 'resolve' when an object is filtered. The promise state was incorrectly set as 'reject' before. This kills the process and occasionally prevents the filtered results from being logged to the console.

  • fix: Send the message 'No issues found after custom filtering' to warn. Before this message was sent to stdout resulting in invalid json.

@Torsten1014 Torsten1014 requested a review from a team as a code owner May 24, 2024 15:51
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.

1 participant