Skip to content

feat: add positive and negative thresholds for score change evaluation#98

Open
bjohansebas wants to merge 7 commits intoossf:mainfrom
bjohansebas:diff
Open

feat: add positive and negative thresholds for score change evaluation#98
bjohansebas wants to merge 7 commits intoossf:mainfrom
bjohansebas:diff

Conversation

@bjohansebas
Copy link
Copy Markdown

closes #88

@bjohansebas bjohansebas force-pushed the diff branch 3 times, most recently from f5836dd to f62e381 Compare January 10, 2026 01:14
@bjohansebas
Copy link
Copy Markdown
Author

Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
…te action inputs

Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
@bjohansebas
Copy link
Copy Markdown
Author

Okay, this still doesn’t prevent a pull request from being created when there are no changes in the diff.

… content

Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
…ate database change check

Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
@bjohansebas
Copy link
Copy Markdown
Author

Now it does take the score changes into account.

: reportContent)
const scoreChanges = scores.filter(score => score.currentDiff !== undefined)

if (scoreChanges.length) {
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

With this, we could already make it output whether something changed. Issue #34 would be completed, but I’d prefer to do that in another PR.

@UlisesGascon UlisesGascon self-assigned this Jan 19, 2026
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.

Add threshold to the issues notification

2 participants