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.

Sign up for GitHub

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

Jump to bottom

fix issue #105 arithmetic operator between int and text value #123

Closed
wants to merge 82 commits into from
Closed

fix issue #105 arithmetic operator between int and text value #123

wants to merge 82 commits into from

Conversation

tzzed
Copy link
Contributor

@tzzed tzzed commented Jul 13, 2020

This is a fix of #105

genji> select 1 + 'hello';
{
  "1 + 'hello'": null
}
genji>

touk and others added 30 commits June 7, 2020 13:56
update set allow dotNotation for document and array
d3bcf8e
update set allow dotNotation for document and array
3495af8
@tzzed
update set allow dotNotation for document and array
3ed3ae3
@tzzed
retire fmt.Println
ff2b649
@tzzed @asdine
Update document/document.go
1883368 
Co-authored-by: Asdine El Hrychy <[email protected]>
@tzzed
manage multiple document
c2c7f03
@tzzed
manage multiple document
a7d4628
@tzzed
manage multiple document
efb63ba
@tzzed
through a document almost recursive
c162ae1
@tzzed
through a document almost recursive
cb8ec1c
@tzzed
added document manager more properly
5207c15
@tzzed
added document manager more properly
015c256
@tzzed
Merge branch 'update-setDotN' into develop
aa1f8a9
@tzzed
clean repo
ea7c7c5
@tzzed
update document ok
0cbc987
@tzzed
update replace field
a325cfe
@tzzed
set document ok + units tests
1dacfd8
@tzzed
document ok missing array
41880bf
@tzzed
manage update array, field and documents with errors handlers for man…
fae5b51 
…y cases
@tzzed
fix document, and array
53e60ab
@tzzed
everything ok remain errors cases
1796ffe
@tzzed
add error manager, more tests
24a9279
@tzzed
add error manager, more tests
3036ab4
@tzzed
error in array
18e4894
@tzzed
error in array git commit error
e6fcaec
@tzzed
error set docomentValue
f1b2dc2
@tzzed
error set docomentValue
e2b5f09
@tzzed
broken commit
d881a68
@tzzed
broken commit
15864e9
@tzzed
corrected commit, remain array manager
fc88b68
tzzed added 24 commits July 5, 2020 17:41
@tzzed
more units test, ok, with more refactor, nested more refactor again
bf94205
@tzzed
more units test, ok, with more refactor, nested more refactor again
0f69e74
@tzzed
broken commit
f1e7e32
@tzzed
broken commit
60de387
@tzzed
broken commit
bcaf369
@tzzed
all tests pass, with some refactoring good. Step 1
e08df2e
@tzzed
best version
236ebc2
@tzzed
best version ever
f1ded9e
@tzzed
top version, tests are ok, refactoring too
5421723
@tzzed
top version, tests are ok, refactoring too
131ec98
@tzzed
last version to merge
eb050c7
@tzzed
commit after review of code
7f46596
@tzzed
version to PR
d0658c5
@tzzed
clean git
552bbee
@tzzed
clean git
9dff84c
@tzzed
last commit
3773c60
@tzzed
add merge update setnotation
d016822
@tzzed
revert to master
ab9d196
@tzzed
revert to master
fbe7759
@tzzed
fix arithmetic operator between text and int, unit test ok
9cd187b
@tzzed
fix arithmetic operator between text and int, unit test ok
3a7def2
@tzzed
Merge branch 'fix/operator' of github.com:tzzed/genji into fix/operator
43ff7aa
@tzzed
Merge branch 'fix/operator' of github.com:tzzed/genji into fix/operator
099b8aa
@tzzed
Merge branch 'fix/operator' of github.com:tzzed/genji into fix/operator
5277ab7
@tzzed tzzed changed the title fix arithmetic operator between int and text value fix issue #105 arithmetic operator between int and text value Jul 13, 2020
@tzzed
Copy link
Contributor Author

tzzed commented Jul 13, 2020

test for all operators are sets

@tzzed
Copy link
Contributor Author

tzzed commented Jul 13, 2020

wrote tests for all operators

@tzzed tzzed closed this Jul 14, 2020
@tzzed tzzed mentioned this pull request Jul 15, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tzzed