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

[DEVOPS-616] Added systemWide flag support to task_definition. #96

Merged
merged 2 commits into from
Dec 3, 2024

Conversation

sonnykt
Copy link
Member

@sonnykt sonnykt commented Oct 10, 2024

No description provided.

@sonnykt sonnykt added the Needs review Dev complete ready for review label Oct 10, 2024
@sonnykt sonnykt self-assigned this Oct 10, 2024
@sonnykt sonnykt force-pushed the feature/DEVOPS-616_task_definition_systemwide branch 2 times, most recently from 5dd4d0c to 9d4f30b Compare October 10, 2024 04:47
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this file should not be required anymore since we're now using the mutation builder instead, but I did not remove it yet due to not having proper tests for the new dynamic system, so maybe we can leave this in until the tests are done.

@sonnykt sonnykt force-pushed the feature/DEVOPS-616_task_definition_systemwide branch from 9d4f30b to 9f4896e Compare October 11, 2024 05:59
@steveworley
Copy link
Contributor

Hey @sonnykt — are you able to resolve the mere conflict?

@sonnykt
Copy link
Member Author

sonnykt commented Nov 14, 2024

@steveworley the merge conflict has been resolved

@steveworley steveworley merged commit 61efa09 into master Dec 3, 2024
8 checks passed
@steveworley steveworley deleted the feature/DEVOPS-616_task_definition_systemwide branch December 3, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs review Dev complete ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants