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

Remove @salesforce/templates #201

Merged
merged 1 commit into from
Aug 27, 2020
Merged

Remove @salesforce/templates #201

merged 1 commit into from
Aug 27, 2020

Conversation

lcampos
Copy link
Contributor

@lcampos lcampos commented Aug 27, 2020

What does this PR do?

Remove @salesforce/templates from yarn.lock to prevent getting the code from npm when developing the repo

What issues does this PR fix or reference?

@W-8017763@

@lcampos lcampos requested a review from a team as a code owner August 27, 2020 22:26
@lcampos lcampos mentioned this pull request Aug 27, 2020
@brpowell
Copy link

Won't this be added to the lockfile any time we do yarn install? Shouldn't lerna or the linking script be able to handle if you want to develop against the local version of the package?

@xyc
Copy link
Contributor

xyc commented Aug 27, 2020

@brpowell this won't be added to lock file after install, only after sfdx plugins:link. And yes lerna handles the dependencies so we don't need to have it in the lock file

@lcampos lcampos merged commit 4873651 into develop Aug 27, 2020
@lcampos lcampos deleted the lcampos/update-lock-file branch August 27, 2020 22:39
xyc pushed a commit that referenced this pull request Sep 11, 2020
@xyc xyc mentioned this pull request Sep 11, 2020
xyc pushed a commit that referenced this pull request Sep 11, 2020
* Update contribution guide (#190)

* add coding and commit guidelines, start developing doc

* update contributing docs

* update develop doc, update launch config

* Update CONTRIBUTING.md

* Update README.md

* Update coding-guidelines.md

* Update commit-guidelines.md

* Update developing.md

* Update CONTRIBUTING.md

Co-authored-by: Xiaoyi Chen <xchen@salesforce.com>

* Update commit-guidelines.md

* Update developing.md

* Update developing.md

* update contributing docs

* Update developing.md

Co-authored-by: Emily Kapner <ekapner@salesforce.com>

* port #186 (#199)

* Update dev scripts (#200)

@W-8017763@

* Remove @salesforce/templates from yarn.lock (#201)

@W-8017763@

* Lf/new functions template (#205)

* Adding Functions template

* Adding test for the new template

* Removing unnecessary lines and spaces

* Adding Functions template

* Adding test for the new template

* Removing unnecessary lines and spaces

* Removing unnecessary type and tslint line

* Removing unnecessary type

* Update packages/templates/src/templates/project/functions/ScratchDef.json

There is a specific scratch org feature for functions.

Co-authored-by: lcampos <lcamposguajardo@gmail.com>

* I remove code that affects the plugin

* empty commit

Co-authored-by: lcampos <lcamposguajardo@gmail.com>

Co-authored-by: Emily Kapner <ekapner@salesforce.com>
Co-authored-by: lcampos <lcamposguajardo@gmail.com>
Co-authored-by: Lisandro Fernandez <70181900+LisandroFernandezSF@users.noreply.github.com>
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