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

Remove alloc feature #160

Merged
merged 1 commit into from
Aug 8, 2022
Merged

Remove alloc feature #160

merged 1 commit into from
Aug 8, 2022

Conversation

lopopolo
Copy link
Member

@lopopolo lopopolo commented Aug 7, 2022

boba has a required dependency on the alloc crate, but gates the
availability of the alloc crate with the alloc feature.

boba does not build with no default features as a result. The docs
lead me to believe this was intentional, but with new CI steps in #159,
all Artichoke crates must build and pass a docs build with no default
features.

This will require a semver major bump.

@lopopolo lopopolo added the A-crate-features Area: Compile-time features or attributes. label Aug 7, 2022
@lopopolo lopopolo force-pushed the lopopolo/alloc-required-no-feature branch from d598a93 to ef16d8e Compare August 8, 2022 00:59
`boba` has a required dependency on the `alloc` crate, but gates the
availability of the `alloc` crate with the `alloc` feature.

`boba` does not build with no default features as a result. The docs
lead me to believe this was intentional, but with new CI steps in #159,
all Artichoke crates must build and pass a docs build with no default
features.

This will require a semver major bump.
@lopopolo lopopolo force-pushed the lopopolo/alloc-required-no-feature branch from ef16d8e to 84cdd25 Compare August 8, 2022 01:10
@lopopolo lopopolo merged commit 6468aa0 into trunk Aug 8, 2022
@lopopolo lopopolo deleted the lopopolo/alloc-required-no-feature branch August 8, 2022 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-crate-features Area: Compile-time features or attributes.
Development

Successfully merging this pull request may close these issues.

1 participant