Skip to content

feat: only upload if settings have changed (Issue #508)#1470

Open
qjyqjy1 wants to merge 2 commits into
shanalikhan:masterfrom
qjyqjy1:master
Open

feat: only upload if settings have changed (Issue #508)#1470
qjyqjy1 wants to merge 2 commits into
shanalikhan:masterfrom
qjyqjy1:master

Conversation

@qjyqjy1

@qjyqjy1 qjyqjy1 commented Jun 11, 2026

Copy link
Copy Markdown

Short description of what this resolves:

This PR implements a content-based check before uploading settings to Gist. It prevents unnecessary uploads when no changes are detected, as requested in Issue #508.

Changes proposed in this pull request:

  • Added a content comparison check in src/sync.ts before the upload process starts.
  • Modified package.nls.json to include a new localization string for the "no changes detected" info message.
  • If no changes are found between local files and remote Gist files, the upload is skipped and a message is shown to the user.

Fixes: #508

How Has This Been Tested?

  • Verified the comparison logic between local allSettingFiles and remote gistObj.data.files.
  • Ensured the upload process only proceeds if at least one file has a content mismatch.

IssueHunt Summary

Referenced issues

This pull request has been submitted to:


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