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 intomasterfrom
server-side-diff
Open

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

Conversation

@aaron-weisberg-qz
Copy link
Copy Markdown
Contributor

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

  • 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

Copy link
Copy Markdown
Contributor

@mattmelgard-qz mattmelgard-qz left a comment

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