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

Expressions.kt:295: Populate with all Java binary operators #177

Open
0pdd opened this issue Sep 23, 2022 · 4 comments
Open

Expressions.kt:295: Populate with all Java binary operators #177

0pdd opened this issue Sep 23, 2022 · 4 comments

Comments

@0pdd
Copy link
Member

0pdd commented Sep 23, 2022

The puzzle 165-1cf62450 from #165 has to be resolved:

// @todo #165:90m Populate with all Java binary operators

The puzzle was created by Aleksandr Mishin on 22-Sep-22.

Estimate: 90 minutes, role: DEV.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@mximp
Copy link

mximp commented Oct 7, 2022

@yegor256 please assign to @b0r1ngx

@b0r1ngx
Copy link
Contributor

b0r1ngx commented Oct 7, 2022

@includealex Hello, i was watch code around, but want to ask question:
What and where puzzle asking to populate with all Java binary operators?

And one more, isn't it a good practice to declare abstract function (just after #todo) to fast understand what the context of this function, and what puzzle giver wants from this function and from all solution. (maybe, if applicable call this function somewhere, to get more context)

P. S. I don't want to seem like a person who constantly asks questions, but, i understand that sometimes it must be just in that way, in way how you instantiate the puzzle.

@mximp
Copy link

mximp commented Oct 13, 2022

@includealex Can you please advise on the above?

@includealex
Copy link
Member

@b0r1ngx We have to populate with all Java binary operators. For the current moment it is not implemented double check for some TokenNodes (for instance, DoubleAmpersand)

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

No branches or pull requests

4 participants