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

feat!: core7 w/ jsforce-node #359

Merged
merged 13 commits into from
Apr 23, 2024
Merged

feat!: core7 w/ jsforce-node #359

merged 13 commits into from
Apr 23, 2024

Conversation

mshanemc
Copy link
Contributor

@mshanemc mshanemc commented Apr 10, 2024

this is published as a beta npm package. Set the pjson version property back where it belongs before merging

now with 80% less jsforce bloat!

use top-levle export for testSetup (requires some tsconfig changes to support node16 module resolution, which changes import * stuff
@W-15387805@

@peternhale
Copy link
Collaborator

@mshanemc approved, but needs conflicts resolved.

@WillieRuemmele
Copy link
Member

I'll be looking at fixing the streaming client timeout, which might be a breaking change. I'll be working off of this branch so we can bundle them together

@WillieRuemmele
Copy link
Member

FYI : #364

@peternhale peternhale merged commit e52a4c4 into main Apr 23, 2024
9 checks passed
@peternhale peternhale deleted the sm/core7-jsforce-node branch April 23, 2024 13:01
gilgourevitch pushed a commit to gilgourevitch/salesforcedx-apex that referenced this pull request Jun 6, 2024
* test: new top-level testSetup export

* feat!: core7, jsforce-node

BREAKING CHANGE: use new jsforce, core

* test: node16 module imports

* ci: beta release 5.0.0-beta.0

* Wr/timeout apex (forcedotcom#364)

* fix: timeout working, pjson locked, UT failing

* fix: use native clearInterval

* chore: undo pjson change

* test: add UT

* chore: temp beta

* chore: revert to 5.0.0

* fix: allow test to finsih first and not hang

* chore: temp beta

* test: add assertions for test return before timeout

---------

Co-authored-by: Willie Ruemmele <[email protected]>
Co-authored-by: Willhoit <[email protected]>
Co-authored-by: peternhale <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants