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

serverless-raw example language conversion #732

Merged
merged 7 commits into from
Jun 30, 2020

Conversation

zephyrz73
Copy link
Contributor

  • Upgrade the version of netcoreapp to 3.1 in aws-ts-serverless-raw.

  • Converted the aws-ts-serverless-raw to a python project aws-py-serverless-raw with the 3.1 netcoreapp version.

@zephyrz73 zephyrz73 requested a review from justinvp June 27, 2020 01:59
@leezen leezen self-requested a review June 29, 2020 20:14
Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

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

Some suggestions, but LGTM otherwise! 🎉

aws-py-serverless-raw/Pulumi.yaml Outdated Show resolved Hide resolved
aws-py-serverless-raw/Pulumi.yaml Outdated Show resolved Hide resolved
aws-py-serverless-raw/README.md Outdated Show resolved Hide resolved
aws-ts-serverless-raw/app/app.csproj Show resolved Hide resolved
aws-ts-serverless-raw/app/.gitignore Show resolved Hide resolved
aws-py-serverless-raw/__main__.py Outdated Show resolved Hide resolved
aws-py-serverless-raw/__main__.py Outdated Show resolved Hide resolved
aws-py-serverless-raw/__main__.py Outdated Show resolved Hide resolved
aws-py-serverless-raw/README.md Outdated Show resolved Hide resolved
aws-py-serverless-raw/README.md Show resolved Hide resolved
XUANHE ZHOU added 2 commits June 30, 2020 12:07
…sioned_concurrent_executions 3.aws-py-serverless-raw fix Pulumi.yaml
Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

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

Aside from one more minor suggestion, looks good! Nice work!

Comment on lines +16 to +18
## Deploying and running the Pulumi App
1. Install dependencies. In this example we will install them in a virtual environment named `venv`.
```bash
Copy link
Member

Choose a reason for hiding this comment

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

Nit: Add blank lines between the header and bullets. We have a tool that generates documentation tutorial pages on www.pulumi.com based on these examples, and the markdown renderer used to render pages on the website sometimes has trouble if there aren't blank lines between markdown elements.

Suggested change
## Deploying and running the Pulumi App
1. Install dependencies. In this example we will install them in a virtual environment named `venv`.
```bash
## Deploying and running the Pulumi App
1. Install dependencies. In this example we will install them in a virtual environment named `venv`.
```bash

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh I am so sorry. I forgot to click on approval message. I will do that with my second language conversion pull request

@zephyrz73 zephyrz73 merged commit f2c2f77 into master Jun 30, 2020
@pulumi-bot pulumi-bot deleted the zephyrz73/serverless-raw-conversion branch June 30, 2020 20:37
dixler pushed a commit that referenced this pull request Jan 21, 2022
1. Change the netcore version to 3.1
2. Converted the project aws-ts-serverless-raw into aws-py-serverless-raw
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