Pull in the latest safety-web development #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Change-Id: I7e98fdf3aa147ca8c1e14eb0f2bbf508aa610f60
Change-Id: I2f31bcf5f868c237417f17a75dc58d0e62438fba
Change-Id: I1455d874fd158a5e915e48c4d1a57de40b37cbf2
Change-Id: Ib3f5a77d4b36e21b40413092dc42ddf135b30378
Change-Id: Idd3d9458d2682560d9b9534a87203aa34ac3ba68
Change-Id: I14e9dfd64aea1b8262a762c3689aa07caac34d18
…nt version of tsetse Change-Id: I542eafad8b6570612592af12f43041c7767fc921
…it pulled for tsetse Change-Id: Ice0292972ac65701676e34289688dd03d110cb37
Change-Id: I541078c4ff8cb51f177f6e73242c0637db47aa2b
Change-Id: If68828cde13c7a70276115493e6d311cf489aad8
This implementation is taken from the tsec eslint plugin. It only works with projects that have a tsconfig setup. Change-Id: Ia8c000f96e10ec0a91e9c232af47fee72e590b53
Change-Id: I8b39d9537ca903b26f2a2758ac9c50d3054364df
WIP: this helper will be refined to focus on certain fields and make the test less rigid. Change-Id: I5b0f64c6b52f7b89b7e94bd5c5fbf62af709097b
Change-Id: I4031da14bad7d9d4b8999727e81bd0178b33067c
Change-Id: Ieff5925cfd6ecfbc314dd69ad3d077cab0320ae1
Change-Id: Ic649c4eadd8c036727e7f71af8a1368878d0a4bc
Since test/ was added to the tsconfig, lib.index.js moved to lib/src/index.js. This was caught in the integration tests, but I haven't set the CI yet and I forgot to run them before the tsconfig change. Change-Id: Id9bacc21976090270a008abcfb93c46e3ac01bdb
Change-Id: Ic96d5a7e791bc0e80ea6b7ba5973a03f3a314659
- Canonicalize the finding reports to compare them - Also add the license header - Also add the javascript test project to the integration tests Change-Id: I6eded1df866bdb0fe3a364e2834d361c67a36df1
There are a few breaking changes between ESLint 8 and 9 so it's worth testing that safety-web works with both versions. Change-Id: Ic97ab5d5d7a958631cb2ef527e1748752ed2528b
Change-Id: Icf62c861dc3f95377e10a4474da188214dd01ce9
…ces run x` Change-Id: I4aced5d5fbe69aec917dbf52cf14bb81e2822fe4
Change-Id: Iba160f4b696ce2bfc2bdbd5d3b375c46cc5b865d
Change-Id: Ic594a1ed16102e987bb2e47b6d30d0f524134d52
Also explicitely use the node_module path for running eslint8 as there's a bug in yarn that makes it run the wrong binary yarnpkg/yarn#8590 Change-Id: I0ffb28f7831b8c016d1f6f00b9af9909d78647ed
Change-Id: I17239dc278005c054d83b7e45599bb72205f05d9
Change-Id: I4b111226d5920c0de3ac9ff5236c68277fa5e184
Change-Id: Ide643796658c061627191d0e020750dd721b3cc0
Change-Id: I9a8758a69f39569181864851a7f6dff19fde9543
yarn run eslint doesn't resolve the correct version by default. This was already working, but was not checked. Change-Id: I74fd24254d7ce0f17ad984668c78cf3ab869d127
Change-Id: I55c8548c6ee85d97f7d5d71e08642178d978d704
Also upgrade the version of some dependencies, commands and documentation to match the new Yarn syntax. Change-Id: I21aaa6ba30185fc7c49b31efbcce31660fff7717
Change-Id: Ibcb3320de668abccb68fece5242c74913d4b36e2
aaronshim
approved these changes
Jul 25, 2024
neuracr
added a commit
that referenced
this pull request
Jul 29, 2024
Change-Id: Ia347551b61b61a162712e17b31352051104a646d
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Development setup.
Yarn Berry.
Tests.