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

chore: Update release candidate #5315

Merged
merged 91 commits into from
Jun 13, 2024

Conversation

mgyorke
Copy link
Contributor

@mgyorke mgyorke commented Jun 13, 2024

Release Candidate

PeterSchafer and others added 30 commits April 22, 2024 09:45
When a Snyk CLI command is expected to exit 0 in acceptance tests, it
would be useful for debugging to see the stderr from the process.

This adds an option to runSnykCLI (logErrors) to do this, opt-in. That
way we don't pollute the log with expected non-zero exits in tests that
expect them.
… targeting .NET Standard frameworks (#5169)

* fix: .NET runtime resolution testing now supports projects targeting .NET Standard

* fix: added missing restore file
* feat(ls): update dependency version

- Improve cache handling
- Add support for IDE themes
- Implement CSP in Code Issue HTML template

* fix(deps): upgrade Snyk LS to latest

---------

Co-authored-by: Luke Watts <luke@snyk.io>
* fix(ci): add deployment steps to release branches

* fix(ci): don't run tests on release branches
…o go (#5173)

* chore: migrate upgrade snyk go dependencies to go

Our primary languages for the CLI are Typescript and Go. Whilst
Python is useful for scripting it introduces unnecessary overhead
for contributors. Reducing the number of languages required reduces
maintenance costs in the longer term.

* chore: introduce validation for repo name

* chore: remove -u flag

* chore: include additional information about target commit

* chore: ignore script due to false positive

* refactor: log error and simplify defer usage
* fix(test): add missing tool version and informationUri

* fix: add semanticVersion and adapt tests
…ng (#5205)

* fix: update setup-node github action to v4

* fix: remove node 14 from smoke tests

* chore: add node 20 to smoke test matrix
Co-authored-by: Avishagp <noreply@snyk.io>
* chore: move cleanup of Release Notes to prepare-release

This will enable release notes for preview and rc versions

* chore: use customized template for Release Notes

* chore: ignore conventional-changelog-conventionalcommits
* chore: create release notes for fips versions as well

* chore: upload release notes to s3

* chore: add commit hash to version for rc and preview

* Fix: add missing persist to workspace
- Use RFC 3339 for all timestamps in sbom test result
- Introduced by should pick the first occurrence for human readable sbom
  test.
* fix(sbom): Retain error code during SBOM generation

* fix(cli): Print message of errors wrapping exec.ExitError
Update with UX improvement on unscannable projects and fixed a bug where the .deps file could not be located in certain cases.
* chore: upgrade to newer windows runner

* fix: use linux based hello-world image in windows runners UNIFY-128

---------

Co-authored-by: adrobuta <alexandra.drobut@snyk.io>
* feat: parse data from output workflow

* fix: avoid overwriting err

* test: inline helper function as it is no longer reusable

* fix: fix shadow declaration

* chore: use new GAF version

* chore: remove sausage finger typo

* feat(whoami): switch from experimental endpoint to ga version

---------

Co-authored-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>
#5181)

* feat: [OSM-1040] Added pnpm support under 'enablePnpmCli' feature flag
cmars and others added 22 commits June 4, 2024 08:31
Instead of fetching the goreleaser config from github in release scripts, get
it from the local Go module cache. The snyk-ls package at the git commit
version has already been downloaded to the cache during a build during Go CLI
compilation.
In #5285 the script which extracts the LS protocol version from snyk-ls was
changed to use the Go module cache. `go install` is used as an alternative to
fetching directly from Github.

This requires Go to be installed during the deployment release build.

Adding more observability to the release script.
* fix: Send all instrumentation data from report workflow

* chore: update gaf

* chore: update gaf

* chore: update final gaf version
* chore: use convenience method to get category vector

* chore: update gaf

* chore: update final gaf
…5295)

* test(instrumentation): adds accpetance testing for instrumentation

* chore: linting
feat: set target-reference value in the container monitor request UNIFY-124
* chore: introduce isUnstableVersion() to toggle features

* chore: fix linter

* chore: update gaf

* chore: update gaf

* chore: update naming

* chore: update log header

* chore: fix test

* test: add integration test for preview features enabled

* chore: update to final gaf version
Co-authored-by: Avishagp <noreply@snyk.io>
* chore: add missing gh depency

* chore: ensure release management works without build
Copy link
Contributor

github-actions bot commented Jun 13, 2024

Warnings
⚠️

Please make changes to snyk help text in Gitbook. Changes will be automatically synchronised to Snyk CLI as a scheduled PR.
For more information, see: help/README.md.

⚠️

Since the CLI is unifying on a standard and improved tooling, we're starting to migrate old-style imports and exports to ES6 ones.
A file you've modified is using either module.exports or require(). If you can, please update them to ES6 import syntax and export syntax.
Files found:

  • src/cli/args.ts
  • src/cli/commands/monitor/index.ts
  • src/lib/plugins/get-deps-from-plugin.ts
  • src/lib/plugins/get-single-plugin-result.ts
  • src/lib/project-metadata/target-builders/git.ts
  • test/fixtures/sbom/npm-sbom-test-response.json
  • test/tap/cli-monitor.acceptance.test.ts
  • test/tap/monitor-target.test.ts
⚠️

"refactor(language-server): Replace tab-based with arrow-based navigation (#5312)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"feat(language-server): increase protocol version to 12 [IDE-376] (#5304)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix: Fix issues of type 'Cannot find module ...' in snyk-docker-plugin (#5301)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix(instrumentation): use cli runtime info when translating v1 data (#5302)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"test(instrumentation): acceptance testing instrumentation (CLI-347) (#5295)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"feat(instrumentation): integrate analytics report workflow CLI-302 (#5259)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"feat(code): return exit code 3 for unsupported projects [CLI-309] (#5264)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"feat(logging): Redact additional types of sensitive data from debug logs (#5254)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"feat(language-server): add --all-projects flag scans by default [IDE-318] (#5247)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix(auth): Autodetect IDE usage and fallback to API token based authentication (#5241)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix: [OSM-1007] fixing a bug causing .NET beta scanning to fail on older versions of .NET (#5228)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"Merge pull request #5225 from snyk/feat/support---print-graph-with-mvn-plugin" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"feat: Support user specified repo url for the experimental code test implementation (#5234)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"feat: [OSM-1040] Added pnpm support under 'enablePnpmCli' feature flag (#5181)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix(ls): remove incorrect /v1 path and upgrade consistent ignores scan (#5214)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix(ci): Remove node 14 from github actions and add node 20 for testing (#5205)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"chore: migrate upgrading of snyk go dependencies from python script to go (#5173)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix(iac): Upgrade iac custom rules to address Vulnerabilities [IAC-2944] (#5191)" is too long. Keep the first line of your commit message under 72 characters.

⚠️

"fix: [OSM-1016] .NET runtime resolution testing now supports projects targeting .NET Standard frameworks (#5169)" is too long. Keep the first line of your commit message under 72 characters.

Generated by 🚫 dangerJS against 903f612

@mgyorke mgyorke marked this pull request as ready for review June 13, 2024 16:03
@mgyorke mgyorke requested a review from a team as a code owner June 13, 2024 16:03
Copy link
Contributor

@PeterSchafer PeterSchafer left a comment

Choose a reason for hiding this comment

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

LGTM

@PeterSchafer PeterSchafer merged commit 2872c6f into release-candidate Jun 13, 2024
9 checks passed
@PeterSchafer PeterSchafer deleted the tmp/1718294133-release-candidate branch June 13, 2024 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet