Skip to content

chore(ci): copy generated folder to each repository #301

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

Merged
merged 7 commits into from
Mar 28, 2022
Merged

Conversation

eunjae-lee
Copy link
Contributor

🧭 What and Why

Changes included:

  • cp -r src/ dest with execa doesn't work as intended. It works locally, but on the GitHub Action, it copies src to the inside of dest.
  • Using fs-extra makes it reliable.
  • Also, now we empty first, before copying (in case of deleted files)

@netlify
Copy link

netlify bot commented Mar 28, 2022

Deploy Preview for api-clients-automation canceled.

Name Link
🔨 Latest commit 61316f9
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/6241ced80a678e00096ce6d1

@algolia-bot
Copy link
Collaborator

algolia-bot commented Mar 28, 2022

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the generated/main branch.

@eunjae-lee eunjae-lee marked this pull request as ready for review March 28, 2022 14:18
@eunjae-lee eunjae-lee requested review from a team, damcou and shortcuts and removed request for a team March 28, 2022 14:18
Comment on lines +209 to +210
await emptyDir(tempGitDir);
await copy(clientPath, tempGitDir, { preserveTimestamps: true });
Copy link
Member

Choose a reason for hiding this comment

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

Much better!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants