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

fix(tooling): update vscode devcontainer image #18455

Merged
merged 3 commits into from
Feb 4, 2022
Merged

Conversation

RichiCoder1
Copy link
Contributor

The latest tag of the jsii/superchain image has long been deprecated. This aligns the devcontainer image with the one used for gitpod.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

The `latest` tag of the `jsii/superchain` image has long been deprecated. This aligns the devcontainer image with the one used for gitpod.
@gitpod-io
Copy link

gitpod-io bot commented Jan 15, 2022

@pahud
Copy link
Contributor

pahud commented Jan 19, 2022

Agree and LGTM. Did you try it in your VSCode local and if possible, GitHub Codespaces as well?

@RichiCoder1
Copy link
Contributor Author

Agree and LGTM. Did you try it in your VSCode local and if possible, GitHub Codespaces as well?

Both VSCode Local and GH Codespaces are showing the correct versions of Node and NPM (e.g. the same as gitpod)

@pahud
Copy link
Contributor

pahud commented Jan 19, 2022

Works great in my Codespaces as well. I think it's ok to approve and merge. cc @skinny85 wdyt?

Copy link
Contributor

@skinny85 skinny85 left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @RichiCoder1!

@skinny85 skinny85 added the pr-linter/exempt-test The PR linter will not require test changes label Feb 4, 2022
@mergify
Copy link
Contributor

mergify bot commented Feb 4, 2022

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-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 6d6060b
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify mergify bot merged commit 28647f7 into aws:master Feb 4, 2022
@mergify
Copy link
Contributor

mergify bot commented Feb 4, 2022

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).

@RichiCoder1 RichiCoder1 deleted the patch-1 branch February 5, 2022 05:22
kornicameister added a commit to kornicameister/aws-cdk that referenced this pull request Feb 6, 2022
* origin/master: (27 commits)
  chore(eks): deprecate older versions of EKS (aws#18842)
  fix(tooling): update vscode devcontainer image (aws#18455)
  chore: npm-check-updates && yarn upgrade (aws#18832)
  chore(docs): Fix broken md links (aws#18384)
  chore(lambda-layer-awscli): install awscli with pip and requirements.txt (aws#18800)
  fix(aws-appsync): Strip unsupported characters from Lambda DataSource (aws#18765)
  feat(cfnspec): cloudformation spec v55.0.0 (aws#18827)
  docs(cfnspec): update CloudFormation documentation (aws#18826)
  chore(cxapi): plugin context provider limited by cx schema (aws#18709)
  feat(iotevents): add grant method to Input class (aws#18617)
  chore(cx-api): break circular dependencies (aws#18767)
  docs(core): clarify that `addOverride` does not change property casing (aws#18687)
  feat(s3-deployment): deploy data with deploy-time values (aws#18659)
  docs(cfnspec): update CloudFormation documentation (aws#18808)
  feat(cli): `cdk diff` works for Nested Stacks (aws#18207)
  docs(cfnspec): update CloudFormation documentation (aws#18783)
  chore(lambda-layer-awscli): add update mechanism for AWS CLI (aws#18780)
  chore(release): 1.143.0
  feat(fsx): add support for FSx Lustre Persistent_2 deployment type (aws#18626)
  feat(amplify): support performance mode in Branch (aws#18598)
  ...
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this pull request Feb 21, 2022
The `latest` tag of the `jsii/superchain` image has long been deprecated. This aligns the devcontainer image with the one used for gitpod.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-linter/exempt-test The PR linter will not require test changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants