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

fix: cloud release ci #4336

Merged
merged 3 commits into from
Nov 23, 2023
Merged

fix: cloud release ci #4336

merged 3 commits into from
Nov 23, 2023

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Nov 23, 2023

πŸ€–[deprecated] Generated by Copilot at 7ea386a

Summary

πŸ›πŸ—‘οΈπŸ”§

This pull request removes an unused module from the sealos cloud product and fixes a YAML syntax error in a workflow file. It updates .github/workflows/frontend.yml to exclude the providers/objectstorage module and corrects the indentation of the release-arm-offline-tar job in .github/workflows/cloud-release.yml.

Oh we're the crew of the sealos cloud
And we work on the code all day
We fix the yaml and we drop the objectstorage
And we push the changes away

Walkthrough

  • Remove object storage feature from sealos cloud product (link, link)
  • Fix indentation of release-arm-offline-tar job in .github/workflows/cloud-release.yml (link)

Signed-off-by: yy <lingdie.yy@outlook.com>
Signed-off-by: yy <lingdie.yy@outlook.com>
@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Nov 23, 2023

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 1032
βœ… Successful 363
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 668
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

Copy link

sweep-ai bot commented Nov 23, 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.
  • Apply: Add docstrings to all functions and file headers.

@lingdie lingdie merged commit 10ec576 into labring:main Nov 23, 2023
31 of 32 checks passed
@lingdie lingdie deleted the fix-ci branch November 23, 2023 03:38
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