Skip to content

feat: Add progress logging and summary to script pull#216

Merged
KadirBalku merged 6 commits intoviur-framework:mainfrom
sveneberth:feat/script-pull-report
May 6, 2026
Merged

feat: Add progress logging and summary to script pull#216
KadirBalku merged 6 commits intoviur-framework:mainfrom
sveneberth:feat/script-pull-report

Conversation

@sveneberth
Copy link
Copy Markdown
Member

Show per-file status ([new], [ok], [diff], [forced]) while pulling and print a summary (new/updated/skipped/unchanged) at the end.


Requires #215 to be merged

sveneberth and others added 4 commits May 6, 2026 00:08
…ompt

Replace hash-based comparison with difflib to fix a bug where differing
line endings (\n vs \r\n) between local and server caused every script to
appear changed. splitlines() normalizes line endings before comparing.
Additionally, a colored server→local diff is now shown before asking the
user whether to overwrite.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Show per-file status ([new], [ok], [diff], [forced]) while pulling and
print a summary (new/updated/skipped/unchanged) at the end.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sveneberth sveneberth marked this pull request as ready for review May 6, 2026 16:06
@sveneberth sveneberth added feature New feature or request Priority: Medium This issue may be useful, and needs some attention. labels May 6, 2026
@sveneberth sveneberth changed the title fix: Add progress logging and summary to script pull feat: Add progress logging and summary to script pull May 6, 2026
sveneberth and others added 2 commits May 6, 2026 18:09
Compare content via splitlines() (line-ending-neutral) to distinguish
unchanged files from actual updates even when --force is set.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@KadirBalku KadirBalku merged commit 3ce5ad1 into viur-framework:main May 6, 2026
@sveneberth sveneberth deleted the feat/script-pull-report branch May 6, 2026 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request Priority: Medium This issue may be useful, and needs some attention.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants