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

chore(release): 2.98.0 #27294

Merged
merged 19 commits into from
Sep 26, 2023
Merged

chore(release): 2.98.0 #27294

merged 19 commits into from
Sep 26, 2023

Conversation

aws-cdk-automation
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation commented Sep 26, 2023

See CHANGELOG

rix0rrr and others added 18 commits September 22, 2023 15:02
This change halves the load time of `require('aws-cdk-lib')`, from ~600ms to ~300ms on my machine. It also shaves ~8min off running all integ tests (again, on my machine).

It does this by making all loading of JavaScript modules lazy, so they are only loaded just before any symbols from them are actually accessed (instead of loading all modules at startup). This new automatic module replaces the previous manual effort to do something similar, in `lazy-index.ts`.

Note that this transformation (unfortunately) does not affect TypeScript compilation times: the TypeScript compiler is still going to read all `.d.ts` files. To improve interactive TypeScript performance, use `swc` or `ts-node` with type checking disabled. Fully type checking your application using `tsc` will take the amount of time it takes.

## What's the big idea?

Turning this:

```js
const x = require("x");
module.exports.banana = function() {
  return x.hello();
}
```

Into this:

```js
var x = () => { var tmp = require("x"); x = () => tmp; return tmp; };
module.exports.banana = function () {
    return x().hello();
};
```

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Small change to make it possible to disable a Schedule. It basically works the same as in Rule in aws-events.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Various fixes to awslint. Also adds new exclusions to prevent existing lint errors from failing the build - will address these errors in a follow-up. Linter config, including exclusions, has been moved into a new file `awslint.json`. If this file does not exist, the linter will continue reading and updating the config in `package.json`. Fixes #26839.

Linter changes:
- Include symbols from submodules by using e.g. `assembly.allClasses` instead of `assembly.classes`
- Fix FQNs in the construct linter by including the submodule name
- Use `cloudformationResource` tag (added in cdklabs/awscdk-service-spec#538) to store CFN resource ID when generating L1 constructs. This is used in the CFN resource linter.
- Fix `core-types.ts` to use new package names
- Ignore version suffixes when guessing resource names in `packages/awslint/lib/rules/resource.ts`

The change in `aws-apigateway/lib/resource.ts` is an example of a `props-physical-name` fix.

Breakdown of new linter errors:
```
docs-public-apis: 2308
props-default-doc: 214
props-physical-name: 106
attribute-tag: 13
construct-interface-extends-iconstruct: 8
resource-interface-extends-resource: 8
from-method: 8
props-no-cfn-types: 7
ref-via-interface: 5
from-signature: 4
construct-ctor-props-optional: 3
props-no-arn-refs: 3
props-no-any: 2
integ-return-type: 2
module-name: 1
construct-ctor: 1
props-struct-name: 1
construct-ctor-props-type: 1
no-static-import: 1
public-static-props-all-caps: 1
Total: 2697
```

### Tasks
- [x] Push cdklabs/awscdk-service-spec#538 before merging

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…est (#27257)

init-javascript.integtest.js is failing in the main pipeline with the following error:

```
Error: Cannot find module '/tmp/cdk-integ-0iwta1unfroo/new-entrypoing.mjs' 
```

This PR fixes this by correcting the spelling of `new-entrypoing` to `new-entrypoint`

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Reported by GitHub's code quality scanning tool.

Many of these are problems with regular expressions.


----

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

Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 39.0.3 to 39.2.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/tj-actions/changed-files/releases">tj-actions/changed-files's releases</a>.</em></p>
<blockquote>
<h2>v39.2.0</h2>
<h2>What's Changed</h2>
<ul>
<li>Upgraded to v39.1.2 by <a href="https://github.com/tj-actions-bot"><code>@​tj-actions-bot</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1594">tj-actions/changed-files#1594</a></li>
<li>chore(deps): update actions/checkout digest to 8ade135 by <a href="https://github.com/renovate"><code>@​renovate</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1599">tj-actions/changed-files#1599</a></li>
<li>feat: add support for restricting the deleted files output to only deleted directories by <a href="https://github.com/jackton1"><code>@​jackton1</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1601">tj-actions/changed-files#1601</a></li>
<li>Updated README.md by <a href="https://github.com/tj-actions-bot"><code>@​tj-actions-bot</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1602">tj-actions/changed-files#1602</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v39...v39.2.0">https://github.com/tj-actions/changed-files/compare/v39...v39.2.0</a></p>
<h2>v39.1.2</h2>
<h2>What's Changed</h2>
<ul>
<li>Upgraded to v39.1.1 by <a href="https://github.com/tj-actions-bot"><code>@​tj-actions-bot</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1591">tj-actions/changed-files#1591</a></li>
<li>chore(deps): update dependency <code>@​types/node</code> to v20.6.3 by <a href="https://github.com/renovate"><code>@​renovate</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1592">tj-actions/changed-files#1592</a></li>
<li>chore: add warrning message to diff error by <a href="https://github.com/jackton1"><code>@​jackton1</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1593">tj-actions/changed-files#1593</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v39...v39.1.2">https://github.com/tj-actions/changed-files/compare/v39...v39.1.2</a></p>
<h2>v39.1.1</h2>
<h2>What's Changed</h2>
<ul>
<li>Upgraded to v39.1.0 by <a href="https://github.com/tj-actions-bot"><code>@​tj-actions-bot</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1583">tj-actions/changed-files#1583</a></li>
<li>chore: move variable declaration by <a href="https://github.com/jackton1"><code>@​jackton1</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1584">tj-actions/changed-files#1584</a></li>
<li>chore: Update test.yml by <a href="https://github.com/jackton1"><code>@​jackton1</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1586">tj-actions/changed-files#1586</a></li>
<li>docs: add V0lantis as a contributor for bug by <a href="https://github.com/allcontributors"><code>@​allcontributors</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1587">tj-actions/changed-files#1587</a></li>
<li>Updated README.md by <a href="https://github.com/tj-actions-bot"><code>@​tj-actions-bot</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1588">tj-actions/changed-files#1588</a></li>
<li>fix: error with test by <a href="https://github.com/jackton1"><code>@​jackton1</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1589">tj-actions/changed-files#1589</a></li>
<li>fix: changed_keys and modified_keys output to handle json and escape_json inputs by <a href="https://github.com/V0lantis"><code>@​V0lantis</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1585">tj-actions/changed-files#1585</a></li>
<li>chore: add codacy config by <a href="https://github.com/jackton1"><code>@​jackton1</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1590">tj-actions/changed-files#1590</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a href="https://github.com/V0lantis"><code>@​V0lantis</code></a> made their first contribution in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1585">tj-actions/changed-files#1585</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v39...v39.1.1">https://github.com/tj-actions/changed-files/compare/v39...v39.1.1</a></p>
<h2>v39.1.0</h2>
<h2>What's Changed</h2>
<ul>
<li>Upgraded to v39.0.3 by <a href="https://github.com/tj-actions-bot"><code>@​tj-actions-bot</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1577">tj-actions/changed-files#1577</a></li>
<li>chore(deps): lock file maintenance by <a href="https://github.com/renovate"><code>@​renovate</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1578">tj-actions/changed-files#1578</a></li>
<li>chore(deps): update typescript-eslint monorepo to v6.7.2 by <a href="https://github.com/renovate"><code>@​renovate</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1580">tj-actions/changed-files#1580</a></li>
<li>feat: add support for returning YAML keys for paths that have changed by <a href="https://github.com/jackton1"><code>@​jackton1</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1581">tj-actions/changed-files#1581</a></li>
<li>Updated README.md by <a href="https://github.com/tj-actions-bot"><code>@​tj-actions-bot</code></a> in <a href="https://redirect.github.com/tj-actions/changed-files/pull/1582">tj-actions/changed-files#1582</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v39...v39.1.0">https://github.com/tj-actions/changed-files/compare/v39...v39.1.0</a></p>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a href="https://github.com/tj-actions/changed-files/blob/main/HISTORY.md">tj-actions/changed-files's changelog</a>.</em></p>
<blockquote>
<h1>Changelog</h1>
<h1><a href="https://github.com/tj-actions/changed-files/compare/v39.1.2...v39.2.0">39.2.0</a> - (2023-09-22)</h1>
<h2>🚀 Features</h2>
<ul>
<li>Add support for restricting the deleted files output to only deleted directories (<a href="https://github.com/tj-actions/changed-files/commit/e6ce728d79e6c15a69a0eea43b1968c2f9da2e8b">e6ce728</a>)  - (Tonye Jack)</li>
</ul>
<h2>🐛 Bug Fixes</h2>
<ul>
<li>Fixed test for windows
(<a href="https://github.com/tj-actions/changed-files/commit/e94da5a065ad8dc8e519802967e6e6ccea3f5e87">e94da5a</a>)  - (Tonye Jack)</li>
<li>Fixed the test
(<a href="https://github.com/tj-actions/changed-files/commit/a721d001dd7be793f80500eb3f60ea9db0743864">a721d00</a>)  - (Tonye Jack)</li>
</ul>
<h2>➕ Add</h2>
<ul>
<li>Added missing changes and modified dist assets.
(<a href="https://github.com/tj-actions/changed-files/commit/8af3110d4dfd8c1cc180fee0f14bcf402b598fb0">8af3110</a>)  - (GitHub Action)</li>
<li>Added a test files
(<a href="https://github.com/tj-actions/changed-files/commit/920856cfdd4b4be17810e34b197596397473adf6">920856c</a>)  - (Tonye Jack)</li>
<li>Added missing changes and modified dist assets.
(<a href="https://github.com/tj-actions/changed-files/commit/8296c33fd0180e60f8f7231f51d246bed82abf1f">8296c33</a>)  - (GitHub Action)</li>
<li>Added missing changes and modified dist assets.
(<a href="https://github.com/tj-actions/changed-files/commit/23985514152c809eeea6e96f26c98b66dc77befa">2398551</a>)  - (GitHub Action)</li>
<li>Added missing changes and modified dist assets.
(<a href="https://github.com/tj-actions/changed-files/commit/ff65504325c3dc10f03100fd228c04eeca7cca3a">ff65504</a>)  - (GitHub Action)</li>
<li>Added missing changes and modified dist assets.
(<a href="https://github.com/tj-actions/changed-files/commit/2325baada3477c4ca5bd25dce29d88b1efdf9f01">2325baa</a>)  - (GitHub Action)</li>
</ul>
<h2>➖ Remove</h2>
<ul>
<li>Deleted the test3 directory
(<a href="https://github.com/tj-actions/changed-files/commit/cd1e384723e4d1a184568182ac2b27c53ebf017f">cd1e384</a>)  - (Tonye Jack)</li>
<li>Deleted a single test file
(<a href="https://github.com/tj-actions/changed-files/commit/a52f8621d26d5d9f54b80f74bda2d9eedff94693">a52f862</a>)  - (Tonye Jack)</li>
<li>Removed unused code
(<a href="https://github.com/tj-actions/changed-files/commit/c98e6d20fc234e1f4b0047cd43af2a37f8f38f44">c98e6d2</a>)  - (Tonye Jack)</li>
<li>Removed unused code
(<a href="https://github.com/tj-actions/changed-files/commit/060b3b9b039f423591f1c790691e6d4ddabb1f5e">060b3b9</a>)  - (Tonye Jack)</li>
</ul>
<h2>🔄 Update</h2>
<ul>
<li>Updated README.md (<a href="https://redirect.github.com/tj-actions/changed-files/issues/1602">#1602</a>)</li>
</ul>
<p>Co-authored-by: jackton1 <a href="mailto:[email protected]">[email protected]</a> (<a href="https://github.com/tj-actions/changed-files/commit/8238a4103220c636f2dad328ead8a7c8dbe316a3">8238a41</a>)  - (tj-actions[bot])</p>
<ul>
<li>Update test
(<a href="https://github.com/tj-actions/changed-files/commit/d2486b470f2ea6ffb0bc55a779968d797a2e3009">d2486b4</a>)  - (Tonye Jack)</li>
<li>Updated test
(<a href="https://github.com/tj-actions/changed-files/commit/e7fd9e59a4c7d9b50cd73b32576bf580517e7f60">e7fd9e5</a>)  - (Tonye Jack)</li>
</ul>

</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/tj-actions/changed-files/commit/8238a4103220c636f2dad328ead8a7c8dbe316a3"><code>8238a41</code></a> Updated README.md (<a href="https://redirect.github.com/tj-actions/changed-files/issues/1602">#1602</a>)</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/aaf4339aa1cd7bd2fbe6d869601e3c52baac24ca"><code>aaf4339</code></a> Merge pull request <a href="https://redirect.github.com/tj-actions/changed-files/issues/1601">#1601</a> from tj-actions/feat/add-support-for-restricting-the...</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/d2486b470f2ea6ffb0bc55a779968d797a2e3009"><code>d2486b4</code></a> Update test</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/e7fd9e59a4c7d9b50cd73b32576bf580517e7f60"><code>e7fd9e5</code></a> Updated test</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/e3cc088a9606acc8b7a64528de485e4ae1f868ec"><code>e3cc088</code></a> Merge branch 'main' into feat/add-support-for-restricting-the-deleted-files-o...</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/72d7089b7241aaa6c60dc39e8336f9aef5ebd1da"><code>72d7089</code></a> chore(deps): update actions/checkout digest to 8ade135</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/e94da5a065ad8dc8e519802967e6e6ccea3f5e87"><code>e94da5a</code></a> Fixed test for windows</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/a721d001dd7be793f80500eb3f60ea9db0743864"><code>a721d00</code></a> Fixed the test</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/8af3110d4dfd8c1cc180fee0f14bcf402b598fb0"><code>8af3110</code></a> Added missing changes and modified dist assets.</li>
<li><a href="https://github.com/tj-actions/changed-files/commit/a351a301d2a539ab0e3949e7d8ba1aa8fe7fc145"><code>a351a30</code></a> Merge 62a23f82b163be6455968457151fa0094ec92a63 into 0b947ed818f8a396b19a26354...</li>
<li>Additional commits viewable in <a href="https://github.com/tj-actions/changed-files/compare/76c4d81a6acd339b55bd7407a016981c853eb702...8238a4103220c636f2dad328ead8a7c8dbe316a3">compare view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=tj-actions/changed-files&package-manager=github_actions&previous-version=39.0.3&new-version=39.2.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)


</details>
FEATURE_FLAGS.md is not updated so updated it with rebuild.

----

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

Previously a type from a submodule with a FQN like `aws_service.CfnTheResource.SubType` was not identified as an L1 type, and subjected to stricter L2 rules.

Fixes failing integration tests in https://github.com/cdklabs/awscdk-service-spec

E.g. https://github.com/cdklabs/awscdk-service-spec/actions/runs/6293221129/job/17097529390

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
With this, GitHub will not show them by default in diffs. The diff can still be viewed, but only on request.

Tested on this PR: https://github.com/aws/aws-cdk/pull/27278/files

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
AWS Service Spec packages to latest versions.
…tabase cluster from a snapshot (under feature flag) (#27174)

This PR fixes a bug where an extra database secret is being generated when an RDS database cluster is being created from a snapshot.

On the `DatabaseClusterFromSnapshotProps` interface, we deprecated the `credentials` property and, at the same, introduced `snapshotCredentials` as the recommended replacement. However, the default behavior associated with the `credentials` property was not removed as doing so would introduce a breaking change for some users as detailed in this [PR](#20777). As a result, users just using the recommended `snapshotCredentials` property to create a new RDS database cluster are seeing an extra, unwanted secret being created.

Closes #23815

----

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

----

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

This fixes #27259 by reverting the offending commit.
@aws-cdk-automation aws-cdk-automation added auto-approve pr/no-squash This PR should be merged instead of squash-merging it labels Sep 26, 2023
@aws-cdk-automation aws-cdk-automation requested a review from a team September 26, 2023 13:24
@github-actions github-actions bot added the p2 label Sep 26, 2023
CHANGELOG.v2.md Outdated Show resolved Hide resolved
@aws-cdk-automation
Copy link
Collaborator Author

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: e12f8ed
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

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

@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2023

Thank you for contributing! Your pull request will be automatically updated and merged without squashing (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit b04f852 into v2-release Sep 26, 2023
12 of 13 checks passed
@mergify mergify bot deleted the bump/2.98.0 branch September 26, 2023 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-approve p2 pr/no-squash This PR should be merged instead of squash-merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants