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

Profile Builder: add report for remove_disable_flag at the top level #496

Merged
merged 15 commits into from
Jun 27, 2022
Merged

Profile Builder: add report for remove_disable_flag at the top level #496

merged 15 commits into from
Jun 27, 2022

Conversation

taylorfturner
Copy link
Contributor

  • add report() where needed and changed call from self._profile[i].profile --> self._profile[i].report
  • added tests

@taylorfturner taylorfturner added Work In Progress Solution is being developed New Feature A feature addition not currently in the library labels Jun 24, 2022
@taylorfturner taylorfturner requested a review from micdavis June 24, 2022 18:14
@taylorfturner taylorfturner requested a review from JGSweets as a code owner June 24, 2022 18:14
@taylorfturner taylorfturner self-assigned this Jun 24, 2022
@taylorfturner taylorfturner added the High Priority Dramatic improvement, inaccurate calculation(s) or bug / feature making the library unusable label Jun 27, 2022
@taylorfturner taylorfturner removed the Work In Progress Solution is being developed label Jun 27, 2022
JGSweets
JGSweets previously approved these changes Jun 27, 2022
@JGSweets JGSweets enabled auto-merge (squash) June 27, 2022 16:54
@taylorfturner taylorfturner requested a review from JGSweets June 27, 2022 17:03
@JGSweets JGSweets merged commit c627d7d into capitalone:main Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
High Priority Dramatic improvement, inaccurate calculation(s) or bug / feature making the library unusable New Feature A feature addition not currently in the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants