tools: skip running test-shared on deps changes

[aduh95]

Running test-shared.yml for changes on folder that it doesn't use is pointless, it can't affect it as those do not even make it to the slim tarball it uses:

node/Makefile

Lines 1221 to 1239 in 6176222

	$(RM) -r $(TARNAME)/deps/corepack
	$(RM) $(TARNAME)/test/parallel/test-corepack-version.js
	ifeq ($(SKIP_SHARED_DEPS), 1)
	$(RM) -r $(TARNAME)/deps/ada
	$(RM) -r $(TARNAME)/deps/brotli
	$(RM) -r $(TARNAME)/deps/cares
	$(RM) -r $(TARNAME)/deps/icu-small
	$(RM) -r $(TARNAME)/deps/icu-tmp
	$(RM) -r $(TARNAME)/deps/llhttp
	$(RM) -r $(TARNAME)/deps/nghttp2
	$(RM) -r $(TARNAME)/deps/ngtcp2
	find $(TARNAME)/deps/openssl -maxdepth 1 -type f ! -name 'nodejs-openssl.cnf' -exec $(RM) {} +
	find $(TARNAME)/deps/openssl -mindepth 1 -maxdepth 1 -type d -exec $(RM) -r {} +
	$(RM) -r $(TARNAME)/deps/simdjson
	$(RM) -r $(TARNAME)/deps/sqlite
	$(RM) -r $(TARNAME)/deps/uv
	$(RM) -r $(TARNAME)/deps/uvwasi
	$(RM) -r $(TARNAME)/deps/zlib
	$(RM) -r $(TARNAME)/deps/zstd

[nodejs-github-bot]

Review requested:

• @nodejs/actions

[legendecas]

npm is not a shared library but it is not bundled into node binary as well.

Suggested change

	- deps/ngtcp2/**
	- deps/ngtcp2/**
	- deps/npm/**

[richardlau]

But wouldn't it be better to still check npm still works when Node.js is built with shared libraries?

[aduh95]

I'm fine either way, both of you are making valid points