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

#191. Allow passing of request object directly #193

Merged
merged 2 commits into from
Nov 29, 2023

Conversation

dorner
Copy link
Contributor

@dorner dorner commented Nov 28, 2023

What? Why?

Fixes #191. This allows passing the request object directly without first converting it to a hash.

How was it tested?

Unit tests.

Copy link
Member

@splittingred splittingred left a comment

Choose a reason for hiding this comment

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

Excellent, thanks! Can you add a short note to CHANGELOG.md describing the changes?

@dorner
Copy link
Contributor Author

dorner commented Nov 29, 2023

Added!

@splittingred splittingred merged commit c522a00 into bigcommerce:main Nov 29, 2023
@dorner dorner deleted the 191-direct-request branch November 29, 2023 16:13
@dorner
Copy link
Contributor Author

dorner commented Nov 29, 2023

Awesome, thanks so much! Not sure what the timelines are for releases - when do you think I can expect a version bump? 😁

@splittingred
Copy link
Member

@dorner
Copy link
Contributor Author

dorner commented Nov 29, 2023

YAHOO! Thanks so much!

@splittingred
Copy link
Member

We don't have specified timelines for this library, mainly because it's mostly feature-complete and doesn't get a lot of changes more than maintenance at this point (gruf does a "small-core" model, which allows for plugins/hooks/interceptors to attach into it, allowing us to keep the core nimble and quick). I typically look at what's pending, and decide from there if we're ready for a minor or patch release accordingly.

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.

Allow gruf to receive request objects in call
2 participants