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

TypeScript ECR Documentation Update #3273

Closed
1 of 5 tasks
Funkmyster opened this issue Jul 10, 2019 · 0 comments · Fixed by #5428
Closed
1 of 5 tasks

TypeScript ECR Documentation Update #3273

Funkmyster opened this issue Jul 10, 2019 · 0 comments · Fixed by #5428
Assignees
Labels
@aws-cdk/aws-ecr Related to Amazon Elastic Container Registry docs/guide Related to the developer guide feature-request A feature should be added or improved. good first issue Related to contributions. See CONTRIBUTING.md

Comments

@Funkmyster
Copy link

Note: for support questions, please first reference our documentation, then use Stackoverflow. This repository's issues are intended for feature requests and bug reports.

  • I'm submitting a ...

    • 🪲 bug report
    • 🚀 feature request
    • 📚 construct library gap
    • ☎️ security issue or vulnerability => Please see policy
    • ❓ support request => Please see note at the top of this template.
  • What is the current behavior?
    If the current behavior is a 🪲bug🪲: Please provide the steps to reproduce
    https://docs.aws.amazon.com/cdk/api/latest/docs/aws-ecr-readme.html has
    repository.addLifecycleRule({ maxImageAgeDays: 30 });
    but using maxImageAgeDays returns an error.

Argument of type '{ maxImageAgeDays: number; }' is not assignable to parameter of type 'LifecycleRule'. Object literal may only specify known properties, and 'maxImageAgeDays' does not exist in type 'LifecycleRule'.

  • What is the expected behavior (or behavior of feature suggested)?
    repository.addLifecycleRule({ maxImageAge: cdk.Duration.days(30) });

  • What is the motivation / use case for changing the behavior or adding this feature?

  • Please tell us about your environment:

    • CDK CLI Version: 0.39.0 (build c3a3c88)
    • Module Version: @aws-cdk/aws-ecr: 0.39.0
    • OS: [ AmazonLinux ]
    • Language: [ TypeScript ]
  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. associated pull-request, stackoverflow, gitter, etc)

@eladb eladb added the @aws-cdk/aws-ecr Related to Amazon Elastic Container Registry label Aug 12, 2019
@RomainMuller RomainMuller assigned ghost and rix0rrr and unassigned ghost Aug 26, 2019
@NGL321 NGL321 added docs/guide Related to the developer guide and removed documentation labels Sep 5, 2019
@SomayaB SomayaB added the feature-request A feature should be added or improved. label Nov 11, 2019
@rix0rrr rix0rrr added the good first issue Related to contributions. See CONTRIBUTING.md label Nov 27, 2019
@mergify mergify bot closed this as completed in #5428 Dec 16, 2019
mergify bot pushed a commit that referenced this issue Dec 16, 2019
* docs(ecr): fix deprecated requirement maxImageAgeDays

- on [Automatically clean up repositories](https://docs.aws.amazon.com/cdk/api/latest/docs/aws-ecr-readme.html\#automatically-clean-up-repositories) of Amazon ECR docs, there is deprecated property using on addLifecycleRule example
- due to introduce of Duration Class(#2857), it should be maxImageAge rather than maxImageAgeDays

* chore(ecr): fix deprecated requirement maxImageAgeDays

- maxImageAgeDays were deprecated since #2857, but still used on comments.
- should be chaged to maxImageAge
- fixes #3273
ed-at-work pushed a commit to ed-at-work/aws-cdk that referenced this issue Dec 17, 2019
* docs(ecr): fix deprecated requirement maxImageAgeDays

- on [Automatically clean up repositories](https://docs.aws.amazon.com/cdk/api/latest/docs/aws-ecr-readme.html\#automatically-clean-up-repositories) of Amazon ECR docs, there is deprecated property using on addLifecycleRule example
- due to introduce of Duration Class(aws#2857), it should be maxImageAge rather than maxImageAgeDays

* chore(ecr): fix deprecated requirement maxImageAgeDays

- maxImageAgeDays were deprecated since aws#2857, but still used on comments.
- should be chaged to maxImageAge
- fixes aws#3273
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-ecr Related to Amazon Elastic Container Registry docs/guide Related to the developer guide feature-request A feature should be added or improved. good first issue Related to contributions. See CONTRIBUTING.md
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants