Issue 3555 - Remove audit-ci from dependencies #6056

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.

Sign up for GitHub

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

Jump to bottom

Merged

droideck merged 2 commits into 389ds:main from droideck:remove_audit_ci

Jan 29, 2024

Member

droideck commented Jan 27, 2024

Description: We use npx for audit-ci runs. Hence we don't need the package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: ?


          Issue 3555 - Remove audit-ci from dependencies

b326fee

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: 389ds#3555

Reviewed by: ?

droideck mentioned this pull request

Issue 6043, 6044 - Enhance Rust and JS bundling and add SPDX licenses for both #6045

Merged

vashirov requested changes

View reviewed changes

Member

vashirov left a comment

On a system where audit-ci was not previously installed, make -f rpms stops with

npx audit-ci
Need to install the following packages:
audit-ci@6.6.1
Ok to proceed? (y)

We should update Makefile.am and rpm.mk to use npx --yes audit-ci instead.


          Add --yes flag to npx

e2b7db2

droideck requested a review from vashirov

January 29, 2024 16:48

vashirov approved these changes

View reviewed changes

droideck merged commit 539bb0f into 389ds:main

droideck added a commit that referenced this pull request


          Issue 3555 - Remove audit-ci from dependencies (#6056)

2bd4534

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: @vashirov (Thanks!)

droideck added a commit that referenced this pull request


          Issue 3555 - Remove audit-ci from dependencies (#6056)

ca252e8

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: @vashirov (Thanks!)

droideck added a commit that referenced this pull request


          Issue 3555 - Remove audit-ci from dependencies (#6056)

b6c65f8

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: @vashirov (Thanks!)

droideck added a commit that referenced this pull request


          Issue 3555 - Remove audit-ci from dependencies (#6056)

d1a24ff

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: @vashirov (Thanks!)

droideck added a commit that referenced this pull request


          Issue 3555 - Remove audit-ci from dependencies (#6056)

c8810cc

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: @vashirov (Thanks!)

droideck added a commit that referenced this pull request


          Issue 3555 - Remove audit-ci from dependencies (#6056)

2dc008e

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: @vashirov (Thanks!)

droideck added a commit that referenced this pull request


          Issue 3555 - Remove audit-ci from dependencies (#6056)

0577a11

Description: We use npx for audit-ci runs. Hence we don't need the
package installed at all.
Remove audit-ci from package.json and a new generate package-lock.json.

Related: #3555

Reviewed by: @vashirov (Thanks!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet