-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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(batch): update README #8214
Conversation
This PR is a continuance of a prior PR that was closed and unrecoverable. This PR should continue off the last set of comments/review with @iliapolo seen here: https://github.com/aws/aws-cdk/pull/6931/files/d7598c69b07c79f6a1e8f09ab9d4361a581d1379 |
@stephnr why is the previous PR not recoverable? Can't you re-open? I'd really like to stick to the original one since it contained a lot of comments... |
@stephnr Should I start reviewing this? |
I have finished making all the changes as you recommended @iliapolo. Again, I do apologize for the delay 🙇 |
I did force pushes and then rolled back using my local reflog in git. But GitHub didn't recognize any of the changes even after I made them and pushed. |
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
This should be the final docs update based on the recent changes to:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license