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

[Infra updates] Copy res model infra updates to the condo model #5

Closed
Tracked by #175
jeancochrane opened this issue Oct 20, 2023 · 0 comments · Fixed by #9
Closed
Tracked by #175

[Infra updates] Copy res model infra updates to the condo model #5

jeancochrane opened this issue Oct 20, 2023 · 0 comments · Fixed by #9
Assignees

Comments

@jeancochrane
Copy link
Contributor

We have a number of issues in the backlog to make it easier to deploy and run the residential model:

Once these changes have been deployed to the res model and we're feeling confident in their stability and their usefulness, we should replicate them in the condo model as well.

There may be opportunities for factoring out some shared code into a shared composite action, but that would have the downside of requiring us to manage a third repo containing the action that would need to be updated and versioned in order to make any changes. It may be simpler to just duplicate the logic between these two repos, unless A) the logic is nearly 100% identical or B) we realize that we'll want to use such an action in other repos as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant