Skip to content

Creating scripts encode, decode + tests #1

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
Open
wants to merge 45 commits into
base: master
Choose a base branch
from
Open

Creating scripts encode, decode + tests #1

wants to merge 45 commits into from

Conversation

@katushane
Copy link

Class: Encode.
ObjectEncode.parse(tokens=[...], mode={1/2})

Class: Decode.
ObjectDecode.parse(string='...')

bukina.e added 25 commits December 19, 2019 16:14
functions for delta-strings (compressing delta-string)
7bbf061
function deltaCompression (sorting array and return array of differen…
c4cff19 
…ces)
selecting modes (simple_string=1, delta_string=2)
4f65c1c
add parameter baseInt ( works "Strings with base of numbers"
c4fdc6a 
)
add function for Delta strings with x-ranges
c1eebf3
fix parameter intBase in decode.js
481acd3
fix files with eslint
1987410
encode to class
504ce26
fix encode to class
db3fb19
decode to class
77cf8c3
add methods with underscore (private methods)
91f6ac4
add methods with underscore (private methods)
8b39ce1
fix decode
6125e72
install dependencies: eslint, mocha
5dbf301
module exports class
6794ce1
module exports Encode
238721d
add test for encode
87bea10
fix index.js
b8b8abd
rules for linter
423bc08
fix by eslint
aac5fd9
rules by eslint
75fa8d0
fix by eslint
2592262
tests passing (besides strings with base of numbers)
6626a06
work with Strings with base of numbers
66b2d34
checking maxLength
d5d801c
@katushane katushane assigned katushane and kewka and unassigned katushane Dec 20, 2019
kewka
kewka previously approved these changes Dec 23, 2019
View reviewed changes
@kewka kewka self-requested a review December 23, 2019 12:02
@kewka kewka dismissed their stale review December 23, 2019 12:05

Disprove

kewka
kewka suggested changes Dec 24, 2019
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kewka kewka kewka requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@kewka kewka

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@katushane @kewka @remort