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

refactor: consolidate periods module types #1302

Merged
merged 2 commits into from
Mar 17, 2025

Conversation

bonjourmauko
Copy link
Member

Depends on #1301

Technical changes

  • Move types from openfisca_core.periods to openfisca_core

@bonjourmauko bonjourmauko added the kind:refactor Refactoring and code cleanup label Nov 19, 2024
@bonjourmauko bonjourmauko requested review from a team November 19, 2024 23:57
@bonjourmauko bonjourmauko self-assigned this Nov 19, 2024
@bonjourmauko bonjourmauko changed the base branch from master to refactor/consolidate-tracers-types November 19, 2024 23:58
@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-tracers-types branch from e56d32d to e18d97c Compare November 20, 2024 00:02
@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-periods-types branch from d4712c1 to b634e1f Compare November 20, 2024 00:03
@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-tracers-types branch from e18d97c to 21f4850 Compare November 21, 2024 06:30
@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-periods-types branch from b634e1f to d865806 Compare November 21, 2024 06:31
@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-tracers-types branch from 21f4850 to a47ef80 Compare March 17, 2025 11:39
@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-periods-types branch from d865806 to 12ab1d9 Compare March 17, 2025 11:46
@bonjourmauko bonjourmauko requested a review from eraviart March 17, 2025 12:08
@benjello
Copy link
Member

@benoit-cty could you have a look I am not qualified for those kind of PRs

@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-tracers-types branch from a47ef80 to 4cf0628 Compare March 17, 2025 14:36
Base automatically changed from refactor/consolidate-tracers-types to master March 17, 2025 14:42
@bonjourmauko bonjourmauko force-pushed the refactor/consolidate-periods-types branch from 12ab1d9 to 7a875ab Compare March 17, 2025 14:43
@bonjourmauko bonjourmauko merged commit 811d5fa into master Mar 17, 2025
29 checks passed
@bonjourmauko bonjourmauko deleted the refactor/consolidate-periods-types branch March 17, 2025 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:refactor Refactoring and code cleanup
Projects
Development

Successfully merging this pull request may close these issues.

2 participants