+
Skip to content
This repository was archived by the owner on Aug 31, 2023. It is now read-only.

chore(ci): resolve main branch ci error #3870

Closed
wants to merge 2 commits into from

Conversation

nissy-dev
Copy link
Contributor

@nissy-dev nissy-dev commented Nov 26, 2022

Summary

I try to fix CI error in main branch. The reason why the tests failed in main branch is that snapshots result is different between Ubuntu and MacOS. So, I created each snapshot results.

Test Plan

I confirmed all workflows passed in my forked repo. see nissy-dev#4

@netlify
Copy link

netlify bot commented Nov 26, 2022

Deploy Preview for docs-rometools canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 61ed5ce
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/63823b8b212ec800087eb9f2

@MichaReiser
Copy link
Contributor

Thank you. I addressed the issue in #3869 by normalizing the snapshots. This has the advantage that it works cross platform (for users with non standard temp directories or platforms with different temp paths)

@nissy-dev
Copy link
Contributor Author

OK 👍 I close this PR

@nissy-dev nissy-dev closed this Nov 26, 2022
@nissy-dev nissy-dev deleted the fix-ci-error-2 branch January 8, 2023 00:31
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
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载