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

Add ECSTargetGroupObject to the configuration reference documentation #4649

Merged

Conversation

ouchi2501
Copy link
Contributor

What this PR does / why we need it:

I added ECSTargetGroupObject as it was not documented.

I didn't know to what extent I should add the document, so I added it only to the latest version first.
Please let me know if you need to add it to docs-dev or any other ver.

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

  • How are users affected by this change:
  • Is this breaking change:
  • How to migrate (if breaking change):

@ouchi2501 ouchi2501 force-pushed the add-doc-ecs-targetgroup-object branch from 8f12547 to 0b16370 Compare November 2, 2023 06:54
Copy link

codecov bot commented Nov 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (dd68c04) 30.61% compared to head (35f1505) 30.61%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4649   +/-   ##
=======================================
  Coverage   30.61%   30.61%           
=======================================
  Files         221      221           
  Lines       25937    25937           
=======================================
  Hits         7941     7941           
  Misses      17348    17348           
  Partials      648      648           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@khanhtc1202
Copy link
Member

@ouchi2501 Nice catch 🙏
Please also update the docs file under /docs-dev with the same content 🙏

@ouchi2501
Copy link
Contributor Author

@khanhtc1202
Thanks for the comment 👍🏼
Also updated /docs-dev.

Copy link
Member

@khanhtc1202 khanhtc1202 left a comment

Choose a reason for hiding this comment

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

Thank you 🙌

@khanhtc1202 khanhtc1202 merged commit fd5af4b into pipe-cd:master Nov 2, 2023
13 checks passed
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Nov 5, 2023
…pipe-cd#4649)

* Add ECSTargetGroupObject to the configuration reference documentation

Signed-off-by: uncle <[email protected]>

* Add docs-dev and Change markdown table formatting

Signed-off-by: uncle <[email protected]>

---------

Signed-off-by: uncle <[email protected]>
@github-actions github-actions bot mentioned this pull request Dec 1, 2023
@github-actions github-actions bot mentioned this pull request Feb 6, 2024
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Feb 12, 2024
…pipe-cd#4649)

* Add ECSTargetGroupObject to the configuration reference documentation

Signed-off-by: uncle <[email protected]>

* Add docs-dev and Change markdown table formatting

Signed-off-by: uncle <[email protected]>

---------

Signed-off-by: uncle <[email protected]>
Signed-off-by: 鈴木 優耀 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants