Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Use server side diff#48

Open
aaron-weisberg-qz wants to merge 6 commits into
masterfrom
server-side-diff
Open

Use server side diff#48
aaron-weisberg-qz wants to merge 6 commits into
masterfrom
server-side-diff

Conversation

@aaron-weisberg-qz

@aaron-weisberg-qz aaron-weisberg-qz commented Jul 19, 2024

Copy link
Copy Markdown
Contributor
  • Local diffing is deprecated in-favor or server-side diffs
  • This also has the benefits of cleaning up the diff output to actually only show apps that have real diffs

@mattmelgard-qz mattmelgard-qz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems good to me, only thing I'm wondering is whether we might want to expose this as an option on the action itself so that we can more easily switch from one to the other if needed

@fcrespofastly

Copy link
Copy Markdown

This would be very useful @aaron-weisberg-qz !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants