Skip to content

generate list of misbehaviours#84

Merged
Erikhu1 merged 5 commits intomainfrom
ta_misbehaviours
Sep 16, 2025
Merged

generate list of misbehaviours#84
Erikhu1 merged 5 commits intomainfrom
ta_misbehaviours

Conversation

@Erikhu1
Copy link
Collaborator

@Erikhu1 Erikhu1 commented Sep 15, 2025

Creates a file "TSF/misbehaviours.md" in the branch save_historical_data containing a list of issues from nlohmann/json satisfying the following criteria:

{
Status: Open,
Label: contains the substring "bug"
}

OR

{
Status: Closed,
Label: contains the substring "bug",
CreatedAt: after 2025-04-11 (corresponding to release date of version v3.12.0 of nlohmann/json)
}

If no open issues, file is still overwritten:

image

If target repo (nlohmann/json) is unreachable, abort and fail workflow:

image

Copy link

@Jonas-Kirchhoff Jonas-Kirchhoff left a comment

Choose a reason for hiding this comment

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

Please remove the unused comment in line 164.

@github-actions github-actions bot added the M label Sep 16, 2025
@Erikhu1
Copy link
Collaborator Author

Erikhu1 commented Sep 16, 2025

Title and subtitles added:
image

@Jonas-Kirchhoff
Copy link

If possible, could you add into the description that the list of bugs and misbehaviours is compiled from the github issues from nlohmann/json and that they link to there?

@Erikhu1
Copy link
Collaborator Author

Erikhu1 commented Sep 16, 2025

updated description:
image

@coveralls
Copy link

coveralls commented Sep 16, 2025

Coverage Status

coverage: 99.186%. remained the same
when pulling dc75019 on ta_misbehaviours
into 53769e6 on main.

# Add title to the output file
echo "# Misbehaviours Report" > $OUTPUT_FILE
echo "" >> $OUTPUT_FILE
echo "This report lists known misbehaviours or bugs of v3.12.0 of the nlohmann/json repository. The misbehaviours are compiled from github issues of the nlohmann/json repository, and links to each corresponding issue." >> $OUTPUT_FILE

Choose a reason for hiding this comment

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

Grammar: Is it "link to" or "links to"; or: Is "link" a verb or a substantive?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good catch!

Copy link

@Jonas-Kirchhoff Jonas-Kirchhoff left a comment

Choose a reason for hiding this comment

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

Please check whether the grammar in line 144 is correct. Both variants should be correct, but have a different meaning.

@Erikhu1 Erikhu1 merged commit 90dd357 into main Sep 16, 2025
82 checks passed
@Erikhu1 Erikhu1 deleted the ta_misbehaviours branch September 16, 2025 13:15
Erikhu1 added a commit that referenced this pull request Nov 14, 2025
* generate list of issues with label containing 'bug'

* check if repo reachable

* add title and subtitles

* expand file description

* fix grammar error
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.

3 participants