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.

Sign up for GitHub

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

Jump to bottom

Rename controlColumns -> measuredColumns in code #116

Closed
lsulak opened this issue Nov 9, 2023 · 0 comments · Fixed by #118
Closed

Rename controlColumns -> measuredColumns in code #116

lsulak opened this issue Nov 9, 2023 · 0 comments · Fixed by #118
Assignees
@lsulak
Labels
Agent Issues touching the agent part of the project enhancement New feature or request Model Changes to the data model this project uses (DTO or internal to Agent/Server) refactoring Improving code quality, paying off tech debt, aligning API, cleanup of unused code Server Issues touching the server part of the project
Milestone
MVP: Atum agent

Comments

@lsulak
Copy link
Collaborator

lsulak commented Nov 9, 2023

Background

Mostly in Scala code, DB will be done already.
@lsulak lsulak added enhancement New feature or request Server Issues touching the server part of the project Agent Issues touching the agent part of the project refactoring Improving code quality, paying off tech debt, aligning API, cleanup of unused code Model Changes to the data model this project uses (DTO or internal to Agent/Server) labels Nov 9, 2023
@lsulak lsulak self-assigned this Nov 9, 2023
@lsulak lsulak added this to the MVP: Atum agent milestone Nov 9, 2023
@lsulak lsulak moved this from 🆕 To groom to 🔖 Sprint in CPS small repos project Nov 9, 2023
@lsulak lsulak mentioned this issue Nov 9, 2023
Merged
@lsulak lsulak moved this from 🔖 Sprint to 🏗 In progress in CPS small repos project Nov 10, 2023
lsulak added a commit that referenced this issue Nov 10, 2023
@lsulak
#116: controlCol -> measuredCol
6ab4763
@lsulak lsulak moved this from 🏗 In progress to 👀 In review in CPS small repos project Nov 10, 2023
@lsulak lsulak mentioned this issue Nov 10, 2023
Merged
lsulak added a commit that referenced this issue Nov 12, 2023
@lsulak
#116: measuredCol -> measuredColumn,, for the sake of consistency
8bdcc2e
lsulak added a commit that referenced this issue Nov 12, 2023
@lsulak
#116: controlCol -> measuredCol (#118)
f09fc4d 
* #116: controlCol -> measuredCol

* #116: measuredCol -> measuredColumn,, for the sake of consistency
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in CPS small repos project Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lsulak lsulak

Labels
Agent Issues touching the agent part of the project enhancement New feature or request Model Changes to the data model this project uses (DTO or internal to Agent/Server) refactoring Improving code quality, paying off tech debt, aligning API, cleanup of unused code Server Issues touching the server part of the project
Projects
CPS small repos project
Archived in project
Milestone
MVP: Atum agent
Development

Successfully merging a pull request may close this issue.

#116: controlCol -> measuredCol AbsaOSS/atum-service
1 participant
@lsulak