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

Split Objective into its own module #929

Merged
merged 1 commit into from
Dec 20, 2022
Merged

Split Objective into its own module #929

merged 1 commit into from
Dec 20, 2022

Conversation

kostmo
Copy link
Member

@kostmo kostmo commented Dec 20, 2022 •
edited
Loading

Towards #795 and in preparation for #927, which adds a lot more Objective-related code.

This is a no-op refactoring, code-movement only.

@kostmo
Towards #795
277d359 
In preparation for #927
@kostmo kostmo requested a review from byorgey December 20, 2022 05:10
@kostmo kostmo added the merge me Trigger the merge process of the Pull request. label Dec 20, 2022
@mergify mergify bot merged commit 5543cb2 into main Dec 20, 2022
@mergify mergify bot deleted the objectives-split branch December 20, 2022 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xsebek xsebek xsebek approved these changes

@byorgey byorgey Awaiting requested review from byorgey

Assignees
No one assigned
Labels
merge me Trigger the merge process of the Pull request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kostmo @xsebek