Skip to content
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

Update services.yml #4254

Merged
merged 1 commit into from
Nov 4, 2023
Merged

Update services.yml #4254

merged 1 commit into from
Nov 4, 2023

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Nov 4, 2023

🤖 Generated by Copilot at 692b369

Summary

🗑️🛠️🚀

Simplify services workflow by removing redundant command. This change removes the line sudo rm -rf registry from .github/workflows/services.yml to avoid unnecessary operations and potential errors.

rm -rf registry
No need to repeat yourself
Simplify, avoid

Walkthrough

  • Simplify services workflow by removing redundant command (link)

@sealos-ci-robot
Copy link
Member

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 981
✅ Successful 312
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 668
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

Copy link

sweep-ai bot commented Nov 4, 2023

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.

@lingdie lingdie merged commit 74f9426 into labring:main Nov 4, 2023
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants