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.

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

Sync with Pro #10261

Merged
merged 4 commits into from
May 20, 2024
Merged

Sync with Pro #10261

merged 4 commits into from
May 20, 2024

Conversation

IagoAbal
Copy link
Collaborator 

OSS repo was last synced with Pro commit 413902d50aa48f9877b8ff8dea925d1de5f2e14c.
There are 14 commit(s) to sync.
Syncing...
* 84ea534860e Add PR template (#1522)
* Skipping (empty) 82525da21a6 annotations: Improve error handling (#1499)
* Skipping (empty) 16ea20a246b sync: Fix references to tickets and PRs (#1532)
* Skipping (empty) dbc705fa9d0 chore: remove tracing from benchmarks (#1530)
* Skipping (empty) 45d8448ef5d Adding changelog.d symlink (#1533)
* Skipping (empty) d5eb34f3cbb sync: Add a GHA workflow to trigger syncs manually (#1534)
* Skipping (empty) 6c4353d6203 fix: sync: Regenerate sync-with-OSS.yml (#1538)
* Skipping (empty) 2d9eea4ff55 devops: no need submodules: recursive anymore (#1536)
* Skipping (empty) 80d8f810ef5 github: Copy PR checklist from OSS (#1535)
* 6972a795f27 devops: add oss cli tests (#1524)
* Skipping (empty) bb692ac9429 fix: Extend JS/TS array method propagators (#1527)
* f75eeee464d fix: Track taint through JSX spread (#1523)
* 6e5c9d5e36d chore: Bump version to 1.73.0 (#1545)
* Skipping (empty) 40f73902355 sync: Clone OSS repo using the https URL (#1546)
Success!

aryx and others added 4 commits May 18, 2024 17:50
@aryx @IagoAbal
Add PR template (semgrep/semgrep-proprietary#1522)
8a358b3 
test plan:
wait for github and check if PR has this template

synced from 84ea534860eea2b1f1dbfc02e3116b96f6b038ed
@akuhlens @IagoAbal
devops: add oss cli tests (semgrep/semgrep-proprietary#1524)
1e8a6c7 
Runs the CLI tests from the OSS Repo with the artifact built from this
repo. I disable one test in the CI job which seems to have had a
regression or something. I need to investigate the difference in the
snapshots, but I will do that in a follow-up PR.


PR checklist:
- [x] the commit message does not contain sensitive information (if

synced from 6972a795f2778a014daaf787694a397d3c4d0be7
@nmote @IagoAbal
fix: Track taint through JSX spread (semgrep/semgrep-proprietary#1523)
c0b3d31 
Followup to semgrep/semgrep-proprietary#1510.

The added test case is not found without this change.

Test plan: Automated tests

synced from f75eeee464d9a23c59b8000de71dae167808be7e
@IagoAbal
chore: Bump version to 1.73.0 (semgrep/semgrep-proprietary#1545)
98b9ab4 
Release PR "Release Version 1.73.0" (#10257)

synced from OSS 7f92bd3

Co-authored-by: aryx <aryx@users.noreply.github.com>

synced from 6e5c9d5e36d8ab84ef44b6a40414b438e3bad8e8
@github-actions GitHub Actions
Copy link
Contributor

PR checklist:

  • Purpose of the code is evident to future readers
  • Tests included or PR comment includes a reproducible test plan
  • Documentation is up-to-date
  • A changelog entry was added to changelog.d for any user-facing change
  • Change has no security implications (otherwise, ping security team)

If you're unsure about any of this, please see:

@IagoAbal IagoAbal requested a review from aryx May 18, 2024 15:57
@IagoAbal
Copy link
Collaborator Author

IagoAbal commented May 18, 2024 •
edited

In case you merge this, REMEMBER that it should be merged with "REBASE and merge" (do NOT use "squash").

@aryx aryx merged commit 53b10cd into develop May 20, 2024
54 checks passed
@aryx aryx deleted the sync-with-pro-1716047428 branch May 20, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aryx aryx aryx approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@IagoAbal @aryx @akuhlens @nmote