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

Apply ES2018 syntax uniformly #724

Merged
merged 46 commits into from
May 1, 2020
Merged

Apply ES2018 syntax uniformly #724

merged 46 commits into from
May 1, 2020

Conversation

dgaubert
Copy link
Contributor

No description provided.

@dgaubert
User ES6 class syntax for MapStore
a957d82
@dgaubert
Use default arguments value
3b3b14a
@dgaubert
Rename argument
4e663ea
@dgaubert
Assign arguments to the instance's root
7fb0463
@dgaubert
Siplify usage of the internal pool
80c3021
@dgaubert
Better error handling and logging
fc636ec
@dgaubert
Return error when error so there is no need to log
ce3e188
@dgaubert
Remove private method
ae2f44b
@dgaubert
Add method to compose redis keys
792c6af
@dgaubert
Better defaults
f64f453
@dgaubert
One liner
c37c436
@dgaubert
Apply ES6
06fa097
@dgaubert
ES6 syntax
180bd64
@dgaubert
Same export order among renderers
e0313e5
@dgaubert
Use object instead of a list of arguments
41f5935
@dgaubert
Stop using index.js to export renderer factories
b73fee8
@dgaubert
Simplify constructor
2f5fc58
@dgaubert
Inline method
a13c77c
@dgaubert
ES6 syntax
b47ccf5
@dgaubert
Remove unneeded try/catch
556de80
@dgaubert
Use try/catch where is needed
1f3fbfa
@dgaubert
Use ES6 class syntax
ec97221
@dgaubert
ES6 class syntax
2a63969
@dgaubert
ES6 class syntax
e585888
@dgaubert
ES6
e1724d8
@dgaubert
ES6
fe6fdfe
@dgaubert
ES6
4e53cde
@dgaubert
Stop injecting a psql wrapper into TorqueRenderer to be able to mock …
45a4ccc 
…responses while testing.
@dgaubert
Use ES6 class syntax
c3defd1
@dgaubert
Use ES6 syntax
6bc913b
@dgaubert dgaubert changed the title Es6 Apply ES2018 syntax uniformly May 1, 2020
@dgaubert dgaubert merged commit 1aac151 into master May 1, 2020
@dgaubert dgaubert deleted the es6 branch May 1, 2020 14:08
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
@dgaubert