-
Notifications
You must be signed in to change notification settings - Fork 20
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(sfdx-project): add name property to project schema #35
Conversation
sfdx-project.schema.json
Outdated
"name": { | ||
"title": "name", | ||
"type": "string", | ||
"description": "The name of your SFDX project." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this to "Salesforce project."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 changed
092a33c
to
e94c6e9
Compare
@cafreeman I think this is ok, are you also going to update the documentation around this file ? e.g. https://developer.salesforce.com/docs/atlas.en-us.sfdx_dev.meta/sfdx_dev/sfdx_dev_ws_config.htm, https://developer.salesforce.com/docs/atlas.en-us.sfdx_dev.meta/sfdx_dev/sfdx_dev2gp_config_file.htm We should also open a PR to add this new attribute to the current scaffold commands in the Salesforce CLI (https://github.com/forcedotcom/salesforcedx-templates). |
@lcampos 👋 I've actually already opened up a PR over on the templates repo if you'd like to take a look forcedotcom/salesforcedx-templates#296 |
What does this PR do?
Adds a name property to the
sfdx-project.json
schemaWhat issues does this PR fix or reference?
https://gus.lightning.force.com/one/one.app#/alohaRedirect/a07AH0000009lOcYAI
Functionality Before
Functionality After
Should be no different except that the name field is documented.