Skip to content

Conversation

@nija-at
Copy link
Contributor

@nija-at nija-at commented Nov 18, 2021


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

otaviomacedo and others added 2 commits November 18, 2021 16:32
…t's not used (#3184)

When `add-deprecation-warnings` is `true` but no calls were injected in a given file, the `require` statement for `.warnings.jsii.js` should not be generated.


---

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

[Apache 2.0 license]: https://www.apache.org/licenses/LICENSE-2.0
@nija-at nija-at self-assigned this Nov 18, 2021
@nija-at nija-at changed the base branch from main to release November 18, 2021 16:36
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Nov 18, 2021
@mergify
Copy link
Contributor

mergify bot commented Nov 18, 2021

The title of this Pull Request does not conform with [Conventional Commits] guidelines. It will need to be adjusted before the PR can be merged.
[Conventional Commits]: https://www.conventionalcommits.org

@nija-at nija-at added the pr/no-squash This PR should be merged instead of squash-merging it label Nov 18, 2021
@mergify
Copy link
Contributor

mergify bot commented Nov 18, 2021

Thank you for contributing! ❤️ I will now look into making sure the PR is up-to-date, then proceed to try and merge it!

@mergify mergify bot added the pr/ready-to-merge This PR is ready to be merged. label Nov 18, 2021
@mergify mergify bot merged commit d848ef8 into release Nov 18, 2021
@mergify mergify bot deleted the nija-at/1.44.2 branch November 18, 2021 17:35
@mergify
Copy link
Contributor

mergify bot commented Nov 18, 2021

Merging (no-squash)...

@mergify mergify bot removed the pr/ready-to-merge This PR is ready to be merged. label Nov 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. 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.

2 participants