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

feat: speacial args (from #190) #219

Closed
wants to merge 2 commits into from

Conversation

cafkafk
Copy link
Member

@cafkafk cafkafk commented Sep 10, 2024

I've resolved the conflicts from #190, but I don't wanna force push over it
because it was... pretty involved, and in case I've manage to break
something, having the original state around is pretty nice.

fwiw, in hindsight, I should have just cherry picked
633852a.

...ohh well...

Copy link
Member Author

@cafkafk cafkafk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've done nothing to check this is correct yet, so don't just merge without some assurance.

@cafkafk cafkafk marked this pull request as ready for review September 10, 2024 11:40
@cafkafk
Copy link
Member Author

cafkafk commented Sep 10, 2024

I've done nothing to check this is correct yet, so don't just merge without some assurance.

I.e. I could easily have broke something or special args may not work... I did try pretty hard not do break any conflicts with new features, like 7c94672, 8e01ad8 or https://github.com/DBCDK/morph/pull/214/files

@cafkafk
Copy link
Member Author

cafkafk commented Sep 17, 2024

closed in favor of #222

@cafkafk cafkafk closed this Sep 17, 2024
@cafkafk cafkafk deleted the special-args-conflict-resolution branch September 17, 2024 07:17
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 this pull request may close these issues.

2 participants