Skip to content

✨Remove unsafe rust code#2629

Open
eselimsen wants to merge 3 commits intorelease/9.5from
feat/meta-7111-remove-unsafe/release/9.5
Open

✨Remove unsafe rust code#2629
eselimsen wants to merge 3 commits intorelease/9.5from
feat/meta-7111-remove-unsafe/release/9.5

Conversation

@eselimsen
Copy link
Copy Markdown

Removes unsafe rust usages from windmill and forbids it in cargo config.
Unsafe static accesses are replaced with std::RwLock as those were global settings written only once or a few times, but read often.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


Thank you for your contribution! Before we can merge this PR, we need you to sign our Contributor License Agreement (CLA).

To sign the CLA, please comment on this PR with:
I have read the CLA Document and I hereby sign the CLA

This is a one-time requirement. Once you have signed, all future contributions will be covered.

You can read the full CLA document here: https://github.com/sequentech/step/blob/main/.github/cla/CLA.md


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

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.

1 participant