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

chore: Remove arc-swap from the top-level crate #9538

Merged
merged 1 commit into from
Oct 9, 2021
Merged

Conversation

blt
Copy link
Contributor

@blt blt commented Oct 8, 2021

This commit removes arc-swap from top-level as it is only used in
lib/enrichment. Similar to the work done in #9535 and #9537.

Signed-off-by: Brian L. Troutwine brian@troutwine.us

@netlify
Copy link

netlify bot commented Oct 8, 2021

✔️ Deploy Preview for vector-project ready!

🔨 Explore the source changes: 55d2b08

🔍 Inspect the deploy log: https://app.netlify.com/sites/vector-project/deploys/6161c0427eac1b0008413226

😎 Browse the preview: https://deploy-preview-9538--vector-project.netlify.app

blt added a commit that referenced this pull request Oct 8, 2021
This commit makes zstd an optional dependency, shaving 12 seconds off a
`--no-default-features` build. Discovered while working on #9515. Relates to:

* #9538
* #9537
* #9535

Signed-off-by: Brian L. Troutwine <brian@troutwine.us>
@blt blt force-pushed the no_toplevel_arcswap branch from e7b8444 to c33142a Compare October 9, 2021 00:24
@blt blt enabled auto-merge (squash) October 9, 2021 00:24
blt added a commit that referenced this pull request Oct 9, 2021
This commit makes zstd an optional dependency, shaving 12 seconds off a
`--no-default-features` build. Discovered while working on #9515. Relates to:

* #9538
* #9537
* #9535

Signed-off-by: Brian L. Troutwine <brian@troutwine.us>
blt added a commit that referenced this pull request Oct 9, 2021
This commit makes zstd an optional dependency, shaving 12 seconds off a
`--no-default-features` build. Discovered while working on #9515. Relates to:

* #9538
* #9537
* #9535

Signed-off-by: Brian L. Troutwine <brian@troutwine.us>
This commit removes arc-swap from top-level as it is only used in
lib/enrichment. Similar to the work done in #9535 and #9537.

Signed-off-by: Brian L. Troutwine <brian@troutwine.us>
@blt blt force-pushed the no_toplevel_arcswap branch from c33142a to 55d2b08 Compare October 9, 2021 16:16
@blt blt merged commit 89e2b08 into master Oct 9, 2021
@blt blt deleted the no_toplevel_arcswap branch October 9, 2021 17:31
@StephenWakely
Copy link
Contributor

Ooops. 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