-
Notifications
You must be signed in to change notification settings - Fork 405
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
docs: expanded pack related updates #5596
Conversation
information about expanded pack and code builder related updates.
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.
Made a suggested change for a couple of typos. LGTM!
Co-authored-by: mleonardsfdc <[email protected]>
fixed the setup to say only expanded pack, not extension pack.
Update README for Expanded pack.
update Code Builder Environment Checklist
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.
LGTM, but we may want to get product review on this as well. The "why" of these changes is a little unclear, and I would be confused by it as a developer.
Randi's feedback on links.
Co-authored-by: AnanyaJha <[email protected]>
Ananya's feedback.
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.
Sorry, I misread the setup section and have a revision note for you!
Update based on Randi's engineering updates.
changes in engineering workflow.
updated again per request from @randi274 |
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.
Thanks for going through this one so carefully with me!
CLI telemetry instructions.
Co-authored-by: Randi Wilson <[email protected]>
information about expanded pack and code builder related updates.
What does this PR do?
Update set up topic to reflect moving e4d to expanded pack and its consequences.
What issues does this PR fix or reference?
@W-15735348@
Functionality Before
e4d not in expanded pack
Functionality After
e4d now in expanded pack