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

[6.0.0] codebuild_project - update tags param to use dict rather than list of dicts #1643

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jan 9, 2023

SUMMARY

Passing list of dicts was deprecated, move it to dict

fixes: #1546
Originally deprecated in #1221 as part of the tagging cleanup.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

codebuild_project

ADDITIONAL INFORMATION

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Jan 9, 2023
@tremble tremble added this to the 6.0.0 milestone Jan 9, 2023
@github-actions
Copy link

github-actions bot commented Jan 9, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@tremble tremble force-pushed the deprecation/remove/6.0.0/codebuild_project branch 3 times, most recently from 17032c5 to 33aeba3 Compare January 9, 2023 15:36
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 14s
✔️ build-ansible-collection SUCCESS in 5m 02s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 22s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 01s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 39s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 06s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 09s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 36s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 32s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 20s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 32s
✔️ ansible-test-changelog SUCCESS in 2m 32s
✔️ ansible-test-splitter SUCCESS in 2m 36s
✔️ integration-community.aws-1 SUCCESS in 6m 19s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jan 11, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/ansible-collections/community.aws for 1643,33aeba3068544eac01ae6fae601a376942418a89

@tremble tremble force-pushed the deprecation/remove/6.0.0/codebuild_project branch from 33aeba3 to d38b0c4 Compare January 11, 2023 11:36
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 29s
✔️ build-ansible-collection SUCCESS in 5m 38s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 06s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 49s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 22s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 18s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 38s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 56s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 02s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 45s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 8m 07s
✔️ ansible-test-changelog SUCCESS in 2m 39s
✔️ ansible-test-splitter SUCCESS in 2m 40s
✔️ integration-community.aws-1 SUCCESS in 6m 09s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 47s
✔️ build-ansible-collection SUCCESS in 7m 48s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 18s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 14m 37s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 13m 33s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 15m 43s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 11m 33s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 9m 06s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 52s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 8m 22s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 8m 02s
✔️ ansible-test-changelog SUCCESS in 3m 02s
✔️ ansible-test-splitter SUCCESS in 2m 45s
✔️ integration-community.aws-1 SUCCESS in 6m 57s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 6c66cfc into ansible-collections:main Jan 11, 2023
@tremble tremble deleted the deprecation/remove/6.0.0/codebuild_project branch February 15, 2023 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[6.0.0] codebuild_project - update tags to use a dict instead of a list of dicts
3 participants