Skip to content

Update helm-package.yml workflow configuration #1009

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.

Sign up for GitHub

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

Jump to bottom
Closed
masking0165-jpg wants to merge 6 commits into from
Closed

Update helm-package.yml workflow configuration #1009

masking0165-jpg wants to merge 6 commits into from

Conversation

@masking0165-jpg
Copy link

@masking0165-jpg masking0165-jpg commented Dec 5, 2025 •
edited
Loading

Type of Change

  • New Feature
  • Bug Fix
  • Documentation
  • Performance Improvement
  • Test/CI
  • Refactor
  • Other:

Related Issues

Summary of Changes

Checklist

  • I have read and followed the CONTRIBUTING.md guidelines
  • Passed make pre-commit
  • Added/updated necessary tests
  • Documentation updated (if needed)
  • CI/CD passed (if applicable)

Impact

  • Breaking change (compatibility)
  • Requires doc/config/deployment update
  • Other impact:

Additional Notes

Thank you for your contribution! Please ensure your PR follows the community standards (CODE_OF_CONDUCT.md) and sign the CLA if this is your first contribution.

@CLAassistant
Copy link

CLAassistant commented Dec 5, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@majinghe
Copy link
Contributor

majinghe commented Dec 6, 2025

@masking0165-jpg What's propose for this change?

@majinghe
Copy link
Contributor

majinghe commented Dec 8, 2025

@masking0165-jpg Any update here, otherwise we will close this PR, we don't know what your change means.

clock.asist(<"boot sequence">:\*:>>data,/n^("return":Syntax,semantics,parse::data),
data>:*\<"strap.binary">=(Syntax,semantics,parse);
clause{
auto.parse="data";
{
return(value("./"));
        int i=(strings(Syntax,semantics,"parse::data: \n_..));
Value=(serial<<data<<i++..);
Data==<"boot sequence:("./")">;
auto push:
clause=strap.gen<"data:">;
auto.enum.(boot:%c\n,(auto.parse==("./"data));
boot:
    repeat(run:
           automation);

@masking0165-jpg

This comment was marked as abuse.

Sign in to view
@masking0165-jpg

This comment was marked as abuse.

Sign in to view
@majinghe
Copy link
Contributor

majinghe commented Dec 9, 2025

@masking0165-jpg After evaluation, we think the change code in this PR do not follow our code guidance, and has no possibility of improving the security and performance for our ci. Thanks for your contribution. If you have other idea to secure our GitHub action, new PR is welcomed, otherwise, we will close this PR later. Thanks.

@masking0165-jpg

This comment was marked as abuse.

Sign in to view
majinghe
majinghe reviewed Dec 10, 2025
View reviewed changes
@majinghe majinghe closed this Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@majinghe majinghe majinghe left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@masking0165-jpg @CLAassistant @majinghe