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(cli)!: Rename --UNSAFE to --UNCONFINED #2011

Merged
merged 3 commits into from
Jan 26, 2024

Conversation

rekmarks
Copy link
Contributor

@rekmarks rekmarks commented Jan 26, 2024

Renames the UNSAFE flag to UNCONFINED, which is more specific. Also renames internals using "unsafe" (like import-unsafe) to use "unconfined" instead.

Will require an endo purge -f.

@rekmarks rekmarks requested review from kriskowal and dckc January 26, 2024 00:06
Renames the `UNSAFE` flag to `UNCONFINED`, which is more specific.
@rekmarks rekmarks force-pushed the rekmarks-unconfined-not-unsafe branch from f9aa898 to ba33e82 Compare January 26, 2024 00:09
Copy link
Member

@kriskowal kriskowal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, to cut all the way down to the bone, I would like to rename importUnsafeAndEndow to makeUnconfined and importBundleAndEndow to merely makeBundle throughout the daemon.js. I would also like to rename the formula types, import-unsafe and import-bundle to merely make-unconfined and make-bundle. These changes will necessitate an endo purge --force, but c’est la mode.

Renames all internals that use "unsafe" in their names to
use "unconfined" instead. Will require an `endo purge -f`.
packages/daemon/src/types.d.ts Outdated Show resolved Hide resolved
@rekmarks rekmarks force-pushed the rekmarks-unconfined-not-unsafe branch from 556ffe0 to 5ca7331 Compare January 26, 2024 01:53
@rekmarks rekmarks merged commit 02776b1 into endo Jan 26, 2024
14 checks passed
@rekmarks rekmarks deleted the rekmarks-unconfined-not-unsafe branch January 26, 2024 02:01
@dckc
Copy link
Contributor

dckc commented Jan 26, 2024

an lo! there was great rejoicing!

thanks!

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.

3 participants