Skip to content

Get variable values as int, double and bool #65

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

Merged
merged 2 commits into from
Oct 13, 2024

Conversation

Tandashi
Copy link

@Tandashi Tandashi commented Aug 3, 2022

Allows to get the variable values casted to the specific type. If the variable does not exist, the fallback is used. However, when also no fallback is provided, an AssertionError is thrown. Furthermore, a FormatException is thrown when the variable exists, but the value can not be casted to the specific type.

Allowes to get the variable values casted to the specific type. If the variable does not exist the `fallback` is used. However when also no `fallback` is provided an `AssertionError` is thrown. Furthermore a `FormatException` is thrown when the variable exists but the value can not be casted to the specific type.
@Tandashi Tandashi changed the title Get variable values as int, double and bool Get variable values as int, double and bool Aug 3, 2022
@java-james
Copy link
Owner

Thanks for making a PR for this feature. Have had a skim and looks good will make time for a proper look & a release if merged!

@Tandashi
Copy link
Author

Resolved the merge conflicts :)

Any update on this? Just curious.

@Tandashi
Copy link
Author

@java-james Are there any updates on this / is there anything that still needs to be done before this can be merged?

@java-james
Copy link
Owner

I'm sorry it's taken this long to get eyes back on this.
I will be resolving a few issues and merging a few PRs in the next couple of weeks. Let me include reviewing this.

@java-james
Copy link
Owner

Will include in next release with some docs on usage. Thanks for the feature and apologies for the time it took to get back to this one.

@java-james java-james merged commit 7ded2fa into java-james:master Oct 13, 2024
@Tandashi
Copy link
Author

@java-james All good no worries. Was / am happy to contribute to this :)

@java-james
Copy link
Owner

Thanks. I'm aiming to cut the next release on this library by end of week :)

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