Skip to content

Esm packages #2415

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

Open
wants to merge 40 commits into
base: master
Choose a base branch
from
Open

Esm packages #2415

wants to merge 40 commits into from

Conversation

klocke-io
Copy link
Contributor

What this PR does / why we need it:
Converts packages to esm
Which issue(s) this PR fixes:
Partially addresses #2294

klocke-io and others added 30 commits April 16, 2025 12:58
…cruiser

This reverts commit 578451e

Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
fix import path

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
…rt path file extension

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…igrated rm -rf to a cross-platform script rimraf

Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
klocke-io and others added 10 commits April 30, 2025 15:49
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>
Signed-off-by: klocke-io <niklas.klocke@sap.com>

# Conflicts:
#	diagram/packages-lowlevel-dependency-diagram.html
#	packages/kube-client/lib/groups.js
@gardener-robot
Copy link

@klocke-io You need rebase this pull request with latest master branch. Please check.

@gardener-robot gardener-robot added needs/review Needs review size/xl Size of pull request is huge (see gardener-robot robot/bots/size.py) needs/second-opinion Needs second review by someone else labels Apr 30, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Apr 30, 2025
router.use(bodyParser.json({
limit: config.maxRequestBodySize,
}))
router.use(authenticate({ createClient }))

Check failure

Code scanning / CodeQL

Missing rate limiting High

This route handler performs
authorization
, but is not rate-limited.

Copilot Autofix

AI 4 days ago

Copilot could not generate an autofix suggestion

Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.

@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Apr 30, 2025
@gardener-robot
Copy link

@holgerkoser, @grolu You have pull request review open invite, please check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/rebase Needs git rebase needs/review Needs review needs/second-opinion Needs second review by someone else size/xl Size of pull request is huge (see gardener-robot robot/bots/size.py)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants