Skip to content

chore: pad golden file timestamps #12256

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

Closed
wants to merge 2 commits into from
Closed

Conversation

Emyrk
Copy link
Member

@Emyrk Emyrk commented Feb 21, 2024

Making the replacement the same length keeps table formats the same in a golden file.

@Emyrk Emyrk requested a review from mafredri February 21, 2024 21:59
@Emyrk Emyrk changed the title chore: padding on golden file timestamps chore: pad golden file timestamps Feb 21, 2024
Copy link
Member

@mafredri mafredri left a comment

Choose a reason for hiding this comment

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

Oh, didn't realize we'd mess with the JSON as well, maybe not worth fixing the golden file in this case but I'll leave it up to you. Thanks for trying it out anyway!

@Emyrk
Copy link
Member Author

Emyrk commented Feb 22, 2024

@matifali The json is affected, but it's not terrible?

Making the replacement the same length keeps table formats the same
in a golden file.
@Emyrk Emyrk force-pushed the stevenmasley/golden-spacing branch from 48a100c to ff4d5c1 Compare February 23, 2024 15:29
@Emyrk
Copy link
Member Author

Emyrk commented Feb 23, 2024

I am going to close this since most of our golden files are json.

If the padding becomes a visual issue, we can readdress.

@Emyrk Emyrk closed this Feb 23, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Feb 23, 2024
@github-actions github-actions bot deleted the stevenmasley/golden-spacing branch August 24, 2024 00:05
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.

2 participants