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

Simple JSON validation and display lines with empty authors #120

Merged
merged 1 commit into from
Sep 26, 2024
Merged

Simple JSON validation and display lines with empty authors #120

merged 1 commit into from
Sep 26, 2024

Conversation

igorpecovnik
Copy link
Member

@igorpecovnik igorpecovnik commented Sep 26, 2024 •
edited
Loading

Description

Actions errors out on invalid JSON structure and display warning for empty author lines.

Testing Procedure

  • Verified at PR with valid and broken JSON

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have ensured that my changes do not introduce new warnings or errors
  • No new external dependencies are included
  • Changes have been tested and verified
  • I have included necessary metadata in the code, including associative arrays

@github-actions github-actions bot added size/small PR with less then 50 lines GitHub Actions GitHub Actions code labels Sep 26, 2024
@igorpecovnik igorpecovnik force-pushed the validate-json branch 4 times, most recently from e2b8bbe to e91de69 Compare September 26, 2024 20:01
@github-actions github-actions bot added GitHub GitHub-related changes like labels, templates, ... Scripts labels Sep 26, 2024
@igorpecovnik igorpecovnik force-pushed the validate-json branch 5 times, most recently from 979d5a9 to dfb06a4 Compare September 26, 2024 20:10
@igorpecovnik igorpecovnik changed the title Simple JSON validation Simple JSON validation and display lines with empty authors Sep 26, 2024
@igorpecovnik igorpecovnik added 11 Milestone: Fourth quarter release Ready to merge Reviewed, tested and ready for merge labels Sep 26, 2024
@Tearran Tearran merged commit bd256bc into main Sep 26, 2024
54 of 64 checks passed
@Tearran Tearran deleted the validate-json branch September 26, 2024 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tearran Tearran Tearran approved these changes

Assignees
No one assigned
Labels
11 Milestone: Fourth quarter release GitHub Actions GitHub Actions code GitHub GitHub-related changes like labels, templates, ... Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@igorpecovnik @Tearran