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(wasm): remove bundled datakit filter #14012

Merged
merged 2 commits into from
Dec 12, 2024

Conversation

flrgh
Copy link
Contributor

@flrgh flrgh commented Dec 12, 2024

KAG-6021

see also: #14013 (master branch PR)

It seemed silly to include the dependency bump changelog entry for datakit, so I removed changelog/3.9.0/kong/bump-datakit.yml.

@github-actions github-actions bot added build/bazel cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Dec 12, 2024
@flrgh flrgh added skip-changelog and removed cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Dec 12, 2024
@flrgh
Copy link
Contributor Author

flrgh commented Dec 12, 2024

FYI this has a changelog entry in changelog/3.9.0, but I think the changelog checker only looks in changelog/unreleased.

@flrgh flrgh force-pushed the chore/wasm-unbundle-datakit-3.9.x branch from 39bb5a2 to 353a4ac Compare December 12, 2024 21:00
@pull-request-size pull-request-size bot added size/M and removed size/S labels Dec 12, 2024
@github-actions github-actions bot added the cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee label Dec 12, 2024
locao
locao previously requested changes Dec 12, 2024
Copy link
Contributor

@locao locao left a comment

Choose a reason for hiding this comment

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

Let's remove the new changelog entry before merging.

@kikito kikito added this to the 3.9.0 milestone Dec 12, 2024
@kikito kikito removed the cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee label Dec 12, 2024
@kikito kikito dismissed locao’s stale review December 12, 2024 22:51

We actually did have the experimental in 3.7, so removing it without a changelog entry is a bit iffy

@kikito kikito merged commit c2613e0 into release/3.9.x Dec 12, 2024
30 checks passed
@kikito kikito deleted the chore/wasm-unbundle-datakit-3.9.x branch December 12, 2024 22:51
kikito pushed a commit that referenced this pull request Dec 17, 2024
* chore(wasm): remove bundled datakit filter

* change log level
kikito added a commit that referenced this pull request Dec 18, 2024
* docs(release): generate 3.9.0 changelog (#13935)

* docs(release): generate 3.9.0 changelog

* docs(changelog): apply suggestions for 3.9.0 changelog

* chore(wasm): remove bundled datakit filter (#14012)

* chore(wasm): remove bundled datakit filter

* change log level

* docs(changelog): update 3.9.0 changelog (#14016)

---------

Co-authored-by: Michael Martin <[email protected]>
Co-authored-by: Andy Zhang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants