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 platform code #5687
Remove platform code #5687
Conversation
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. one minor docs question. gonna merge so we can go forward and we can re add this if necessary
@@ -3,7 +3,6 @@ title: Serverless Framework Documentation | |||
menuText: Docs | |||
layout: Doc | |||
menuItems: | |||
- {menuText: Getting Started, path: /framework/docs/getting-started/} |
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.
Any particular reason for removing this?
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.
Yes, it is a duplicate. The docs ALSO create links for each file in the dir. Since there is a getting-started.md
file as well as this metadata link, this resulted in duplicate links on our docs site.
Nope. Good catch @exoego ! |
What did you implement:
Removed the Serverless Platform code from the Framework.
Is this ready for review?: NO
Is it a breaking change?: NO