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

Support Arithmetic function ROUND #594

Closed
shouwn opened this issue Jan 25, 2024 · 2 comments
Closed

Support Arithmetic function ROUND #594

shouwn opened this issue Jan 25, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@shouwn
Copy link
Member

shouwn commented Jan 25, 2024

Backgrounds

  • Kotlin JDSL does not yet support an arithmetic function ROUND.

Objects

  • Support an arithmetic function ROUND.

Requirements

  • Creates round() function in com.linecorp.kotlinjdsl.dsl.jpql.Jpql for supporting an arithmetic function ROUND.

Contribution

  • Please check if there is an assignee in this issue. And if it don't have an assignee, please leave comment to assign.
  • Please check the commit convention through the commitlint.config.js file. The commit convention also described in the PR template.
  • When you use intellij, please set the code style through the .editorconfig file.
  • Please create the PR for develop as the base branch.
  • If you have any questions, please leave comment in Kotlin-JDSL Discord Channel in LINE Open Source
@shouwn shouwn added enhancement New feature or request good first issue Good for newcomers labels Jan 25, 2024
@LeeJejune
Copy link
Contributor

I would like to work on this issue!

@shouwn
Copy link
Member Author

shouwn commented Jan 30, 2024

Hi LeeJejune!

I assigned this issue to you. Thank you for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants