Skip to content

Improve VIDiff: Detect binary files, Add logic for added/deleted VIs#49

Merged
shivaCode-2 merged 10 commits into
mainfrom
users/shivshar/improvement-vidiff
Apr 28, 2026
Merged

Improve VIDiff: Detect binary files, Add logic for added/deleted VIs#49
shivaCode-2 merged 10 commits into
mainfrom
users/shivshar/improvement-vidiff

Conversation

@shivaCode-2
Copy link
Copy Markdown
Collaborator

@shivaCode-2 shivaCode-2 commented Apr 28, 2026

Improvements in the existing VIDiff action:

  1. Added logic to detect binary files by reading the last 8-12 bytes of the file instead of relying on the *.vi filter.
  2. Added logic to generate reports for added/removed VIs as well.
  3. Made the diff comment on the PR better: Now specifies whether the current VI is Added/Deleted/Modified.

Unrelated Change

  1. Modify mirror-to-gitlab action such that its not triggered from forked repo PRs and only runs in base repo main context.
  2. Modified VI Analyzer workflows to upload report as an artifact.

@shivaCode-2 shivaCode-2 force-pushed the users/shivshar/improvement-vidiff branch from 8437ad4 to df359d2 Compare April 28, 2026 22:02
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@ni ni deleted a comment from github-actions Bot Apr 28, 2026
@github-actions
Copy link
Copy Markdown

🔍 VIDiff Reports (linux)

VI Report
Append log (Added) View Report
ArthOps (Modified) View Report
Create output file (Added) View Report
GetHelp (Added) View Report
Make path absolute (Added) View Report
Open VI (Added) View Report
Parse inputs (Added) View Report
Print to temp file (Added) View Report
RunOperation (Added) View Report
base64_fast_encode (Added) View Report

📄 All Reports

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

🔍 VIDiff Reports (windows)

VI Report
Append log (Added) View Report
ArthOps (Modified) View Report
Create output file (Added) View Report
GetHelp (Added) View Report
Make path absolute (Added) View Report
Open VI (Added) View Report
Parse inputs (Added) View Report
Print to temp file (Added) View Report
RunOperation (Added) View Report
base64_fast_encode (Added) View Report

📄 All Reports

@shivaCode-2 shivaCode-2 requested review from Copilot and removed request for Copilot April 28, 2026 22:48
@shivaCode-2 shivaCode-2 merged commit 1c7eff0 into main Apr 28, 2026
10 checks passed
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