New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: show fine-grained detail of changes in npx checkly deploy --preview
#527
Comments
Thanks for checking this out @nahuelon. The two issues are really closely related. We could maybe even tackle them both at once and just completely rework the I created two issues since I want to have the |
@clample can we close this one? |
@tnolet personally I think that it would be a useful feature. It would take a day or two to do, though. Please feel free to close it if you don't think that it's worth doing. |
@clample I mixed this up with the other ticket. We can leave this open. I love the output suggested here. |
@clample @umutuzgur , I finished a first approach displaying a the differences with |
@nahuelon that looks awesome 👀 Should/could we add special handling for browser check code? In your example screenshot for change-1, it could be cool to just show a text diff of the exact line in |
@tnolet maybe you have ideas on how |
@clample @nahuelon the proposal here looks good, I don't think we can fully design up front what will be the most useful layout. Two niggles with the proposed layout:
|
@umutuzgur @clample , we'll need to apply some changes in the backend to be able to compare the |
What problem does this feature solve?
This issue relates to #526, but I think that it's a distinct issue and a bit harder to fix.
Currently
npx checkly deploy --preview
only shows high-level information about the changes: which checks/resources were created, updated, and deleted. It currently doesn't show which fields are changed. For example, it won't show that a check goes fromactivated: true
toactivated: false
. It also doesn't detect the case where a check isn't changed at-all - the check will still show as updated in the diff.It would be great if
npx checkly deploy --preview
gave more fine-grained detail:How would you implement this feature?
Maybe jest-diff is useful. We can also look at
terraform plan
for inspiration on how the output should be formated.The text was updated successfully, but these errors were encountered: