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(tools): add readonly tools.table.EMPTY for common usage #13492

Merged
merged 7 commits into from
Aug 13, 2024

Conversation

chronolaw
Copy link
Contributor

@chronolaw chronolaw commented Aug 12, 2024

Summary

Inspired by #13491.
This PR did not replace all so called EMPTY tables, only which are obviously readonly.

KAG-5140

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

@github-actions github-actions bot added core/clustering core/configuration cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Aug 12, 2024
@chronolaw chronolaw force-pushed the feat/add_tools_empty_table branch from ed997e3 to c2f35ff Compare August 12, 2024 09:43
@chronolaw chronolaw changed the title feat(tools): add tools.table.EMPTY for common usage feat(tools): add readonly tools.table.EMPTY for common usage Aug 12, 2024
Copy link
Contributor

@outsinre outsinre left a comment

Choose a reason for hiding this comment

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

Good, then there is no need to declare {}.

@ADD-SP ADD-SP merged commit 2ffd3b1 into master Aug 13, 2024
31 checks passed
@ADD-SP ADD-SP deleted the feat/add_tools_empty_table branch August 13, 2024 07:13
@team-gateway-bot
Copy link
Collaborator

Cherry-pick failed for master, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git remote add upstream https://github.com/kong/kong-ee
git fetch upstream master
git worktree add -d .worktree/cherry-pick-13492-to-master-to-upstream upstream/master
cd .worktree/cherry-pick-13492-to-master-to-upstream
git checkout -b cherry-pick-13492-to-master-to-upstream
ancref=$(git merge-base 0960f882a944447df02f32390142447d95192d3e 343a3c5254130c24a9cf10985caaf8ba8aaf1ef0)
git cherry-pick -x $ancref..343a3c5254130c24a9cf10985caaf8ba8aaf1ef0

@github-actions github-actions bot added the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Aug 13, 2024
@ADD-SP
Copy link
Contributor

ADD-SP commented Aug 13, 2024

Should be cherry-picked after the https://github.com/Kong/kong-ee/pull/9950 be merged

@kikito kikito removed the incomplete-cherry-pick A cherry-pick was incomplete and needs manual intervention label Aug 13, 2024
oowl pushed a commit that referenced this pull request Aug 15, 2024
Inspired by #13491
This PR did not replace all so-called `EMPTY` tables, only which are obviously read-only.

https://konghq.atlassian.net/browse/KAG-5140
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