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

sorting of issues is not given report.md is off #313

Closed
Tracked by #322
BioPhoton opened this issue Nov 24, 2023 · 0 comments · Fixed by #346
Closed
Tracked by #322

sorting of issues is not given report.md is off #313

BioPhoton opened this issue Nov 24, 2023 · 0 comments · Fixed by #346
Assignees
@MishaSeredenkoPushBased
Labels
🐛 bug something isn't working 🧩 utils
Milestone
2. Integration

Comments

@BioPhoton
Copy link
Collaborator

BioPhoton commented Nov 24, 2023 •
edited by kristihoskova
Loading

Issues should be sorted by severity and then location (line number). Make sure that the sorting logic is not implemented within the scored report. This means sorted in the report .json already.
Screenshot 2023-11-24 at 22 03 23
@BioPhoton BioPhoton added the 🐛 bug something isn't working label Nov 24, 2023
@BioPhoton BioPhoton added this to the 2. Integration milestone Nov 24, 2023
@BioPhoton BioPhoton mentioned this issue Nov 26, 2023
Open
55 tasks
MishaSeredenkoPushBased added a commit that referenced this issue Dec 5, 2023
@MishaSeredenkoPushBased
feat(utils): add sorting of audit issues for report md
c98c152 
Closes #313

#313
Merged
MishaSeredenkoPushBased added a commit that referenced this issue Dec 11, 2023
@MishaSeredenkoPushBased
feat(utils): add sorting of audit issues for report md
b1c5dd7 
Closes #313

#313
BioPhoton pushed a commit that referenced this issue Dec 12, 2023
@MishaSeredenkoPushBased @BioPhoton
feat(utils): add sorting of audit issues for report md
3c06feb 
Closes #313

#313
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MishaSeredenkoPushBased MishaSeredenkoPushBased

Labels
🐛 bug something isn't working 🧩 utils
Projects
None yet
Milestone
2. Integration
Development

Successfully merging a pull request may close this issue.

feat(utils): add sorting of audit issues for report md code-pushup/cli
2 participants
@BioPhoton @MishaSeredenkoPushBased