Skip to content

Conversation

@stephen-carter-at-sf
Copy link
Contributor

Note that we cannot go past:

  • "@oclif/core": "3.27.0",
  • "@salesforce/core": "6.7.6",
  • "@salesforce/sf-plugins-core": "5.0.13",

Mainly because @salesforce/sf-plugins-core v6 is an ESM module and given the nature of how the flags are setup with SfCommand as static property... we can't dynamically import Flags from our commonJS module without a dynamic import - which is not possible at the top level needed for static property.

And because we are stuck on v5 here, this forces us to be stuck on oclif v3 and core v6 due to conflicts if you attempt to upgrade those without also upgrading sf-plugins-core.

Otherwise, I updated everything else, including eslint to v9 (which means we had to move to flat config with compatibility).

}
}
),
...fixupConfigRules(compat.extends("plugin:sf-plugin/recommended")),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is essentially how we keep using the sf-plugin rules even though they haven't migrated to support flat config yet.

Comment on lines +28 to +29
"@eslint/compat": "^1.2.5",
"@eslint/eslintrc": "^3.2.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are needed to maintain the use of the sf-plugin rules which do not support flat config yet... so we need compatibility wrappers.

"@salesforce/core": "^5",
"@salesforce/sf-plugins-core": "^5.0.4",
"@salesforce/core": "6.7.6",
"@salesforce/sf-plugins-core": "5.0.13",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note that since we will be updating these modules regularly now… since JSON doesn’t allow you to add comments, I am basically just removing the ^ prefix for any version that we shouldn’t be updating for whatever reason. This will help us know that there is some issue that we need to look into before attempting to update again. I'll keep notes on the work items... so we can always just look back at them.

@stephen-carter-at-sf stephen-carter-at-sf merged commit ea2e415 into dev Jan 30, 2025
12 checks passed
@stephen-carter-at-sf stephen-carter-at-sf deleted the sc/W-17662327 branch January 30, 2025 19:44
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.

3 participants