Skip to content

Bring back -h#104

Merged
smithgp merged 2 commits into
mainfrom
gps/plugin-help
Apr 13, 2022
Merged

Bring back -h#104
smithgp merged 2 commits into
mainfrom
gps/plugin-help

Conversation

@smithgp
Copy link
Copy Markdown
Collaborator

@smithgp smithgp commented Apr 12, 2022

  • Adds @oclif/plugin-help back, which provides -h
  • Revs @salesforce/command and @salesforce/core
  • Update code for changes to types in those
  • Remove use of deprecated fs from @salesforce/core

smithgp added 2 commits April 12, 2022 16:44
To bring in plugin-help, I also reved @salesforce/core and @salesforce/command.
That then needed some minor updates for changes to the types.
@smithgp smithgp requested a review from reidaelliott April 12, 2022 22:52
@smithgp smithgp self-assigned this Apr 12, 2022
@smithgp smithgp requested a review from a team as a code owner April 12, 2022 22:52
* For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause
*/

import { promises as fs } from 'fs';
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

fs from core is basically the same as fs.promises


import { core } from '@salesforce/command';
import { promises as fs } from 'fs';
import * as core from '@salesforce/core';
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The core import above was never really the right way to import core; this is the right way.

Comment thread tsconfig.json
"rootDir": "src",
"baseUrl": ".",
"paths": {
"@salesforce/core": ["node_modules/@salesforce/core"]
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@salesforce/command re-exports a copy of @salesforce/core, which the typescript type checker picks up first (and causes problems) w/o this line.

Comment thread yarn.lock
@reidaelliott reidaelliott self-requested a review April 13, 2022 15:17
@smithgp smithgp merged commit bf753e2 into main Apr 13, 2022
@smithgp smithgp deleted the gps/plugin-help branch April 13, 2022 16:25
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.

2 participants