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

feat:providers add template provider #3658

Merged
merged 1 commit into from Aug 10, 2023
Merged

Conversation

zjy365
Copy link
Contributor

@zjy365 zjy365 commented Aug 10, 2023

πŸ€– Generated by Copilot at ab5754c

Summary

πŸ“¦πŸŒπŸ› οΈ

This pull request adds a new template provider service to the frontend of the sealos project. The template provider is a Next.js application that can generate dynamic content based on user input and data sources. The pull request also updates the frontend workflow and Dockerfile to support the template provider build and deployment. Additionally, the pull request adds configuration files, translation files, and documentation for the template provider service.

template provider
added to frontend image
a new feature blooms

Walkthrough

  • Add and configure the template provider service for the frontend (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)
  • Install Git and OpenSSH client in the frontend/Dockerfile if the $name argument is template (link)
  • Cache and restore the providers/template directory in the frontend.yml workflow (link, link)
  • Remove the conditional check for the pnpm-lock.yaml file in the frontend/Dockerfile (link)
  • Remove an empty line from the frontend/Dockerfile (link)
  • Add a comma to the typescript dependency in the frontend/package.json (link)

@pull-request-size
Copy link

Whoa! Easy there, Partner!

This PR is too big. Please break it up into smaller PRs.

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Aug 10, 2023

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 885
βœ… Successful 348
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 536
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.15% ⚠️

Comparison is base (81609b2) 67.92% compared to head (97f69ef) 67.77%.
Report is 4 commits behind head on main.

❗ Current head 97f69ef differs from pull request most recent head f674659. Consider uploading reports for the commit f674659 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3658      +/-   ##
==========================================
- Coverage   67.92%   67.77%   -0.15%     
==========================================
  Files           8        8              
  Lines         664      661       -3     
==========================================
- Hits          451      448       -3     
  Misses        171      171              
  Partials       42       42              

see 2 files with indirect coverage changes

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@zjy365 zjy365 force-pushed the desktop branch 4 times, most recently from b6f6c67 to ca36f20 Compare August 10, 2023 06:00
Signed-off-by: jingyang <3161362058@qq.com>
@zjy365 zjy365 marked this pull request as ready for review August 10, 2023 06:23
@c121914yu c121914yu merged commit a62daa6 into labring:main Aug 10, 2023
40 of 43 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.

None yet

3 participants