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

Upgrade all SFDX Commands to SF Commands #3621

Open
GeekStewie opened this issue Jul 17, 2023 · 6 comments
Open

Upgrade all SFDX Commands to SF Commands #3621

GeekStewie opened this issue Jul 17, 2023 · 6 comments
Labels
enhancement Listed under "Changes" in the changelog wi-created This issue is represented by a Work Item

Comments

@GeekStewie
Copy link

Feature Type

I'm interested in functionality that solves a current problem or limitation in how CumulusCI works.

Does your feature request solve a current problem?

I noticed this in the SFDX update today:

IMPORTANT ANNOUNCEMENT, PLEASE READ: We no longer update sfdx (v7); the last stable version is 7.209.6. You can keep using sfdx (v7) if you want, but it won't contain new features or bug fixes. And we no longer update this page.

And looking at the installers, it now defaults to the SF CLI when you install via npm etc.

I know we have had warnings for a while now but looks like the move needs to happen to ensure we are not using unsupported CLIs

Describe the solution you'd like

Migrate all SFDX commands within CumulusCI to SF equivalents

Describe alternatives you've considered

No response

Additional context

No response

@GeekStewie GeekStewie added the enhancement Listed under "Changes" in the changelog label Jul 17, 2023
@jstvz jstvz added the wi-created This issue is represented by a Work Item label Jul 17, 2023
@jstvz
Copy link
Contributor

jstvz commented Jul 17, 2023

Initial work tracked in W-13772594.

@mattmartin26
Copy link

Would really appreciate an update on this, our CTO is considering moving away from CumulusCI depending on if/how soon CCI will transition from SFDX to SF.

@jlantz
Copy link
Contributor

jlantz commented Oct 19, 2023

I'm no longer part of the team, but I do want to clarify the message from @GeekStewie's original post. The deprecation message is that they are no longer maintaining the sfdx package itself. The new sf cli package continues to ship with sfdx though. They just don't want you to install the old sfdx package directly anymore. CumulusCI still works fine against the new sf cli installers since they include sfdx.

The warnings call out a few places where CumulusCI's code needs to be updated to use the new commands and environment variables. I'd love to see them gone to avoid this confusion, but I'm reassured knowing that Salesforce is actively using CumulusCI. The Architecture Decision Record (ADR) discussion for this is available at in PR #3630

CumulusCI is actively built against test projects via GitHub Actions including full packaged builds against a test project. Any failures would be pretty quickly detected by those builds.

@davisagli
Copy link
Contributor

@jlantz It looks like the Github Actions need to be updated to install sf rather than sfdx, though: https://github.com/SFDO-Tooling/CumulusCI/blob/main/.github/workflows/feature_test.yml#L77

@jlantz
Copy link
Contributor

jlantz commented Oct 19, 2023

@davisagli Great point. That's an easy PR that I'll put together shortly as an early warning system at least.

@jstvz
Copy link
Contributor

jstvz commented Oct 19, 2023

@mattmartin26 Thanks for flagging this. Here's a quick update on the SFDX to SF CLI transition:

  • We've started work to migrate SFDX commands in CumulusCI to SF. It's on our radar. For details, check out the discussion on the proposed ADR in ADR: Migrate from SFDX to SF CLI #3630 (feedback welcome).
  • As @jlantz mentioned, the new sf CLI still ships with sfdx, commands will still function as expected despite the warnings.
  • We're adjusting our GitHub Actions per @davisagli's spot, to reflect the ongoing changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Listed under "Changes" in the changelog wi-created This issue is represented by a Work Item
Projects
None yet
Development

No branches or pull requests

5 participants