profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/sikanhe/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

sikanhe/gqtx 197

Code-first type-safe GraphQL Server without codegen or metaprogramming

sikanhe/reason-graphql 138

GraphQL server in pure Reason (Bucklescript)

sikanhe/apollo-tracing-elixir 108

Apollo Tracing middleware for Absinthe

sikanhe/reason-nodejs 96

Node bindings for Reason and Bucklescript

sikanhe/algolia-elixir 81

Elixir implementation of Algolia search API

sikanhe/acme 41

Acme (Let's Encrypt) Client for Elixir

sikanhe/reason-atomic 19

Dead simple shared state in Reason React with a redux-like interface.

sikanhe/reason-dataloader 7

Dataloader in Reason using Future as the async primitive

sikanhe/less-phoenix-example 1

A possible take on phoenix project structure

sikanhe/reasons-game-of-life 1

Conway's game of life in Reason

startedgoogle/zx

started time in 10 hours

PR opened sikanhe/reason-nodejs

Bump hosted-git-info from 2.8.8 to 2.8.9

Bumps hosted-git-info from 2.8.8 to 2.8.9. <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md">hosted-git-info's changelog</a>.</em></p> <blockquote> <h2><a href="https://github.com/npm/hosted-git-info/compare/v2.8.8...v2.8.9">2.8.9</a> (2021-04-07)</h2> <h3>Bug Fixes</h3> <ul> <li>backport regex fix from <a href="https://github-redirect.dependabot.com/npm/hosted-git-info/issues/76">#76</a> (<a href="https://github.com/npm/hosted-git-info/commit/29adfe5">29adfe5</a>), closes <a href="https://github-redirect.dependabot.com/npm/hosted-git-info/issues/84">#84</a></li> </ul> <p><!-- raw HTML omitted --><!-- raw HTML omitted --></p> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/npm/hosted-git-info/commit/8d4b3697d79bcd89cdb36d1db165e3696c783a01"><code>8d4b369</code></a> chore(release): 2.8.9</li> <li><a href="https://github.com/npm/hosted-git-info/commit/29adfe5ef789784c861b2cdeb15051ec2ba651a7"><code>29adfe5</code></a> fix: backport regex fix from <a href="https://github-redirect.dependabot.com/npm/hosted-git-info/issues/76">#76</a></li> <li>See full diff in <a href="https://github.com/npm/hosted-git-info/compare/v2.8.8...v2.8.9">compare view</a></li> </ul> </details> <details> <summary>Maintainer changes</summary> <p>This version was pushed to npm by <a href="https://www.npmjs.com/~nlf">nlf</a>, a new releaser for hosted-git-info since your current version.</p> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+3 -3

0 comment

1 changed file

pr created time in 16 hours

PR opened sikanhe/reason-nodejs

Bump lodash from 4.17.19 to 4.17.21

Bumps lodash from 4.17.19 to 4.17.21. <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/lodash/lodash/commit/f299b52f39486275a9e6483b60a410e06520c538"><code>f299b52</code></a> Bump to v4.17.21</li> <li><a href="https://github.com/lodash/lodash/commit/c4847ebe7d14540bb28a8b932a9ce1b9ecbfee1a"><code>c4847eb</code></a> Improve performance of <code>toNumber</code>, <code>trim</code> and <code>trimEnd</code> on large input strings</li> <li><a href="https://github.com/lodash/lodash/commit/3469357cff396a26c363f8c1b5a91dde28ba4b1c"><code>3469357</code></a> Prevent command injection through <code>_.template</code>'s <code>variable</code> option</li> <li><a href="https://github.com/lodash/lodash/commit/ded9bc66583ed0b4e3b7dc906206d40757b4a90a"><code>ded9bc6</code></a> Bump to v4.17.20.</li> <li><a href="https://github.com/lodash/lodash/commit/63150ef7645ac07961b63a86490f419f356429aa"><code>63150ef</code></a> Documentation fixes.</li> <li><a href="https://github.com/lodash/lodash/commit/00f0f62a979d2f5fa0287c06eae70cf9a62d8794"><code>00f0f62</code></a> test.js: Remove trailing comma.</li> <li><a href="https://github.com/lodash/lodash/commit/846e434c7a5b5692c55ebf5715ed677b70a32389"><code>846e434</code></a> Temporarily use a custom fork of <code>lodash-cli</code>.</li> <li><a href="https://github.com/lodash/lodash/commit/5d046f39cbd27f573914768e3b36eeefcc4f1229"><code>5d046f3</code></a> Re-enable Travis tests on <code>4.17</code> branch.</li> <li><a href="https://github.com/lodash/lodash/commit/aa816b36d402a1ad9385142ce7188f17dae514fd"><code>aa816b3</code></a> Remove <code>/npm-package</code>.</li> <li>See full diff in <a href="https://github.com/lodash/lodash/compare/4.17.19...4.17.21">compare view</a></li> </ul> </details> <details> <summary>Maintainer changes</summary> <p>This version was pushed to npm by <a href="https://www.npmjs.com/~bnjmnt4n">bnjmnt4n</a>, a new releaser for lodash since your current version.</p> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+3 -3

0 comment

1 changed file

pr created time in 21 hours

create barnchsikanhe/reason-nodejs

branch : dependabot/npm_and_yarn/lodash-4.17.21

created branch time in 21 hours

startedjchavarri/hello-jsoo-esy

started time in a day

issue commentrescript-lang/syntax

Cannot parse variant constructor containing object and fat arrow

Thank for the report! Looking into this 👀

sinap-fb

comment created time in 2 days

issue openedrescript-lang/syntax

Cannot parse variant constructor containing object and fat arrow

type queryDelta =
  | Compute(({"blocked_ids": unit}) => unit)

Pop this in the formatter and it reformats to

type queryDelta = Compute({"blocked_ids": unit} => unit)

which gives a syntax error

[E] Line 1, column 48:
I'm not sure what to parse here when looking at "=>".

Thanks!

created time in 2 days

push eventrescript-lang/syntax

Iwan

commit sha 0bc0fd037971b1adb991476af81002cb5f2aa6d1

Tweak `variantIdent` error message based on feedback from bloodyowl Co-authored-by: Matthias Le Brun <bloodyowl@icloud.com>

view details

push time in 2 days

Pull request review commentrescript-lang/syntax

Don't parse Int token with suffices as hash ident for poly variants

 Explanation: since records have a known, fixed shape, a spread like `{a, ...b}` Explanation: lists are singly-linked list, where a node contains a value and points to the next node. `list[a, ...bc]` efficiently creates a new item and links `bc` as its next nodes. `[...bc, a]` would be expensive, as it'd need to traverse `bc` and prepend each item to `a` one by one. We therefore disallow such syntax sugar.\n\ Solution: directly use `concat`." -  let variantIdent = "A polymorphic variant (e.g. #id) must start with an alphabetical letter."+  let variantIdent = "A polymorphic variant (e.g. #id) must start with an alphabetical letter or is a number (e.g. #742)"

Thanks, this sentence definitely needed work.

IwanKaramazow

comment created time in 2 days

pull request commentrescript-lang/syntax

Don't parse Int token with suffices as hash ident for poly variants

Yes, thanks for the suggestion, will do!

IwanKaramazow

comment created time in 2 days

pull request commentrescript-lang/syntax

Don't parse Int token with suffices as hash ident for poly variants

maybe a good time to turn on warning 9 to avoid it in the future?

IwanKaramazow

comment created time in 2 days

Pull request review commentrescript-lang/syntax

Don't parse Int token with suffices as hash ident for poly variants

 Explanation: since records have a known, fixed shape, a spread like `{a, ...b}` Explanation: lists are singly-linked list, where a node contains a value and points to the next node. `list[a, ...bc]` efficiently creates a new item and links `bc` as its next nodes. `[...bc, a]` would be expensive, as it'd need to traverse `bc` and prepend each item to `a` one by one. We therefore disallow such syntax sugar.\n\ Solution: directly use `concat`." -  let variantIdent = "A polymorphic variant (e.g. #id) must start with an alphabetical letter."+  let variantIdent = "A polymorphic variant (e.g. #id) must start with an alphabetical letter or is a number (e.g. #742)"
  let variantIdent = "A polymorphic variant (e.g. #id) must start with an alphabetical letter or be a number (e.g. #742)"
IwanKaramazow

comment created time in 3 days

PR opened rescript-lang/syntax

Don't parse Int token with suffices as hash ident for poly variants

#10s should not be accepted as a numeric polyvariant identifier.

Fixes https://github.com/rescript-lang/syntax/issues/407

+47 -3

0 comment

3 changed files

pr created time in 3 days

create barnchrescript-lang/syntax

branch : GH407

created branch time in 3 days

issue commentrescript-lang/syntax

nulary polymrophic variants starting with leading number are compiled to numbers

Yes, confirmed the bug on my side, should be matching on | Int {i; suffix = None} ->. Int like tokens with a suffix don't qualify as numeric polyvars.

Will make a PR.

amiralies

comment created time in 4 days

issue openedrescript-lang/syntax

nulary polymrophic variants starting with leading number are compiled to numbers

for example

let a = #10s

is compiled to

va a = 10

as result if this if I have a type containing #10s and #10m, these two values are eqivalent

created time in 4 days

issue commentrescript-lang/syntax

Express list literal variants

Something like:

type rec t<'a> =
| Empty
| Cons('a, t<'a>)
rickyvetter

comment created time in 5 days

startedsikanhe/gqtx

started time in 5 days

issue openedrescript-lang/syntax

Remove `{path as p}` record pattern

This formats to {path: p}, which is the right syntax. The as syntax is unnecessarily confusing sugar.

created time in 5 days

pull request commentsikanhe/gqtx

refactor!: merge `defaultField` and `field`

I created https://github.com/FedericoBiccheddu/gqtx/pull/1 for unifying the API with the t.type API as well as handling optional resolve

FedericoBiccheddu

comment created time in 6 days

push eventrescript-lang/syntax

Iwan

commit sha e8479c72e72d84b0262ff7007e883f7bf2e1adfd

Fix printing of Osig_module in outcome printer. (#404) Modules types don't use a `=`, it should be a `:`. ``` module Expr: { … } ```

view details

Cheng Lou

commit sha b89849d8ec7bb66e93d2e3400d53f784b1bb010a

Tiny cleanup

view details

Iwan

commit sha acaedb35b9e131a454a70052bca628b216b019d4

Work

view details

Iwan

commit sha 5257a1cf5d71efc3b6faf42742ac2da1c7d1b4c4

add tests

view details

Iwan

commit sha 72d9a104432b9339da33b90238571f267b8f1e2a

Preserve original char encoding in printer

view details

push time in 6 days

issue commentrescript-lang/syntax

Add support for `res` to `re` syntax conversion

Aside from the technical challenges in a reverse conversion, the above snippet is a bug in refmt. Refmt will print:

let make = (~columns as [@attr] columns, ~rows as [@attr] rows) => ()

as

let make = (~[@attr] columns, ~[@attr] rows) => ()

Refmt doesn't support this syntax, so it's an idempotency problem inside its parser/printer. It's advised to stick to ReScript syntax, both for support and learning purposes. We designed the new syntax to be easier to read and it also features a more responsive typing experience.

jchavarri

comment created time in 6 days

issue closedrescript-lang/syntax

Add support for `res` to `re` syntax conversion

Hi there 👋 Thanks a lot for your work on the ReScript parser :)

We have been testing conversion from res to re syntax, and the generated code seems to be invalid. Here's an example in playground.

The produced code contains attributes in unexpected places:

  let make =
      (
        ~[@ns.namedArgLoc] columns,
        ^^^
  [E] Line 12, column 9:

  We've found a bug for you!
  playground.re:12:10-11
  
  Syntax error

Would there be a way of removing the [@ns.namedArgLoc] from the Reason output? Or are there more attributes / syntax additions in ReScript that would make this conversion hard to implement and support? Even if just for learning purposes in the playground while transitioning to new syntax, maybe there is a use case to support res -> re translations?

closed time in 6 days

jchavarri

issue commentrescript-lang/syntax

Add support for `res` to `re` syntax conversion

Technically, it is impossible to convert the whole res syntax into re and vice versa. (Think about poly var and the forthcoming unicode). It is generally discouraged to write new projects with reason syntax so I am not sure the point in converting res syntax to re, we will eventually remove the reason syntax (that would be a long time though).

jchavarri

comment created time in 6 days

issue openedrescript-lang/syntax

Add support for `res` to `re` syntax conversion

Hi there 👋 Thanks a lot for your work on the ReScript parser :)

We have been testing conversion from res to re syntax, and the generated code seems to be invalid. Here's an example in playground.

The produced code contains attributes in unexpected places:

  let make =
      (
        ~[@ns.namedArgLoc] columns,
        ^^^
  [E] Line 12, column 9:

  We've found a bug for you!
  playground.re:12:10-11
  
  Syntax error

Would there be a way of removing the [@ns.namedArgLoc] from the Reason output? Or are there more attributes / syntax additions in ReScript that would make this conversion hard to implement and support? Even if just for learning purposes in the playground while transitioning to new syntax, maybe there is a use case to support res -> re translations?

created time in 7 days

push eventrescript-lang/syntax

Cheng Lou

commit sha b89849d8ec7bb66e93d2e3400d53f784b1bb010a

Tiny cleanup

view details

push time in 7 days

pull request commentrescript-lang/syntax

Fix printing of Osig_module in outcome printer.

Great. Done.

IwanKaramazow

comment created time in 7 days

pull request commentrescript-lang/syntax

Fix printing of Osig_module in outcome printer.

yes, should be synced with editor + compiler.

IwanKaramazow

comment created time in 7 days

pull request commentrescript-lang/syntax

Fix printing of Osig_module in outcome printer.

Needs syncing to the editor right.

IwanKaramazow

comment created time in 7 days