profile
viewpoint
Marcos Bérgamo thebergamo @epam Gdansk, Poland https://medium.com/@thedon Beer Lover and @nodejs developer

rafaeljesus/koa-bb 15

koa2, jwt, knex, ava and docker boilerplate

thebergamo/appengine-websocket-nodejs 9

Testing the GAE using websocket and node.js

Cart-API/cart 4

Cart API (Node.JS + Hapi + Sequelize Postgres)

rafaeljesus/rest-scheduler 3

Handles job scheduling by storing and managing events that correspond to actions to execute in the future

thebergamo/anton 3

Fantastic Job Queue Worker

Cart-API/cart-ansible 2

Ansible Orquestration for configure a Cart server

issue commentjquense/yup

bug after update 0.31.1 -> 0.32.0

i removed @types/yup but why should i set strictFunctionTypes:false it was working well before

TrejGun

comment created time in 27 minutes

issue commentjquense/yup

bug after update 0.31.1 -> 0.32.0

TypeScript usage is linked in the README. Basically, remove @types/yup, add strictFunctionTypes:false to tsconfig.json, and maybe a couple small adjustments listed in the TypeScript guide.

TrejGun

comment created time in 33 minutes

issue commentjquense/yup

bug after update 0.31.1 -> 0.32.0

The changelog indicates some kind of breaking change but the README is not updated on how to use the new imports.

👎

TrejGun

comment created time in an hour

issue commentjquense/yup

Circular dependency: node_modules/yup/es/index.js -> node_modules/yup/es/mixed.js

I also got this warning

evdama

comment created time in an hour

issue commentjquense/yup

bug after update 0.31.1 -> 0.32.0

Same here, had to rollback

TrejGun

comment created time in 2 hours

issue commentjquense/yup

bug after update 0.31.1 -> 0.32.0

Also had to roll back

TrejGun

comment created time in 2 hours

PR opened nodejs/node

doc: add version metadata to timers/promises

Refs: https://github.com/nodejs/node/pull/33950

Checklist
  • [x] make -j4 test (UNIX), or vcbuild test (Windows) passes
  • [x] documentation is changed or added
  • [x] commit message follows commit guidelines
+9 -0

0 comment

1 changed file

pr created time in 2 hours

issue commentjquense/yup

bug after update 0.31.1 -> 0.32.0

Same here, forced to rollback to 0.31.1 for now.

TrejGun

comment created time in 2 hours

issue commentapollographql/graphql-subscriptions

Authorizing subscriptions

@adri1wald No it's not. subscribe expected to return async iterable not promise, so it would lead to an error.

Check this if you want to do resolver base authorization.

altschuler

comment created time in 2 hours

issue closednodejs/node

Uncaught TypeError: Do not know how to serialize a BigInt

<!-- Thank you for suggesting an idea to make Node.js better.

Please fill in as much of the template below as you're able. -->

Is your feature request related to a problem? Please describe.

JSON.stringify({key:1n});

throws:

Uncaught TypeError: Do not know how to serialize a BigInt
    at JSON.stringify (<anonymous>)

Describe the solution you'd like

"{\"key\":1n}"

Describe alternatives you've considered

when call JSON.parse the string, it should convert back to {key:1n}.

closed time in 2 hours

taoqf

issue commentnodejs/node

Uncaught TypeError: Do not know how to serialize a BigInt

This is working as intended, bigints are not supported by JSON. See https://github.com/tc39/proposal-json-parse-with-source for a WIP solution.

taoqf

comment created time in 2 hours

issue openednodejs/node

Uncaught TypeError: Do not know how to serialize a BigInt

<!-- Thank you for suggesting an idea to make Node.js better.

Please fill in as much of the template below as you're able. -->

Is your feature request related to a problem? Please describe.

JSON.stringify({key:1n});

throws:

Uncaught TypeError: Do not know how to serialize a BigInt
    at JSON.stringify (<anonymous>)

Describe the solution you'd like

"{\"key\":1n}"

Describe alternatives you've considered

when call JSON.parse the string, it should convert back to {key:1n}.

created time in 2 hours

issue openednodejs/node

Illegal instruction: 4

<!-- Thank you for reporting an issue.

This issue tracker is for bugs and issues found within Node.js core. If you require more general support please file an issue on our help repo. https://github.com/nodejs/help

Please fill in as much of the template below as you're able.

Version: output of node -v Platform: output of uname -a (UNIX), or version and 32 or 64-bit (Windows) Subsystem: if known, please specify affected core module name -->

  • Version:14.15.1
  • Platform:MacOS
  • Subsystem:10.11

What steps will reproduce the bug?

launching npm or node <!-- Enter details about your bug, preferably a simple code snippet that can be run using node directly without installing third-party dependencies. -->

How often does it reproduce? Is there a required condition?

it always happens

What is the expected behavior?

<!-- If possible please provide textual output instead of screenshots. -->

What do you see instead?

Illegal instruction: 4 <!-- If possible please provide textual output instead of screenshots. -->

Additional information

<!-- Tell us anything else you think we should know. -->

created time in 3 hours

pull request commentnodejs/node

[v14.x backport] v8: implement v8.takeCoverage() and v8.stopCoverage()

CI: https://ci.nodejs.org/job/node-test-pull-request/34738/

rickyes

comment created time in 3 hours

issue openedjquense/yup

bug after update 0.31.1 -> 0.32.0

Describe the bug types seems to be broken

Screen Shot 2020-12-04 at 9 28 14 AM

To Reproduce

import * as Yup from "yup";

export const validationSchema = Yup.object().shape({
  email: Yup.string(),
});

Expected behavior A clear and concise description of what you expected to happen.

Platform (please complete the following information):

  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Additional context i was using 0.31.1 with @types/yup but 0.32.0 does not need them

created time in 4 hours

startedoguimbal/pg-mem

started time in 4 hours

PR opened nodejs/node

[14.x] stream: fix regression on duplex end

Backport Decide the return status of writeOrBuffer before calling stream.write which can reset state.length

Refs: #35941 Fixes: #35926

Based off @mmomtchev 's work in #36229 , rebased on v14.x-staging

<!-- Thank you for your pull request. Please provide a description above and review the requirements below.

Bug fixes and new features should include tests and possibly benchmarks.

Contributors guide: https://github.com/nodejs/node/blob/master/CONTRIBUTING.md -->

Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

  • [x ] make -j4 test (UNIX), or vcbuild test (Windows) passes
  • [x ] tests and/or benchmarks are included
  • [ x] commit message follows commit guidelines

<!-- Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or

(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or

(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.

(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved. -->

+38 -6

0 comment

2 changed files

pr created time in 4 hours

startedkeroserene/rickrollrc

started time in 4 hours

startedKasperskyLab/TinyCheck

started time in 5 hours

issue commentrust-lang/mdBook

404 page displays without CSS if the address is a subfolder

Do you have a custom theme or are you setting the site-url in the book.toml? Which version of mdbook are you using?

The default template sets a base url which for the 404 page defaults to /.

Simonbdy

comment created time in 5 hours

Pull request review commentrust-lang/mdBook

Guide: Add instructions for publishing via GitLab Pages

 deploy: book 		git commit -m "deployed on $(shell date) by ${USER}" && \ 		git push origin gh-pages ```++## Deploying Your Book to GitLab Pages+Inside your repository's project root, create a file named `.gitlab-ci.yml` with the following contents:+```yml+stages:+    - deploy++pages:+  stage: deploy+  image: rust:latest+  variables:+    CARGO_HOME: $CI_PROJECT_DIR/cargo+  before_script:+    - export PATH="$PATH:$CARGO_HOME/bin"+    - mdbook --version || cargo install --debug mdbook

Any particular reason to use --debug here? On large books, it can be very slow (though I guess on CI it's not too important). It's also quite a bit larger on disk.

FWIW, I usually suggest that CI installs the precompiled binaries (such as here), as it can be substantially faster and avoids dealing with caching issues. I realize this is following the style of the other examples, just putting it out there.

sburris0

comment created time in 5 hours

issue commentnodejs/node

Node.js 14.x supported on windows 8.1 and above?

@eladkarako thanks! In case anyone has problems, this version of node should work fine for Windows 7 if you follow all the great instructions @eladkarako provided. https://nodejs.org/dist/v14.15.1/node-v14.15.1-win-x64.zip

gruber123

comment created time in 5 hours

issue commentjquense/yup

Circular dependency: node_modules/yup/es/index.js -> node_modules/yup/es/mixed.js

@evdama I have this warning too

evdama

comment created time in 5 hours

pull request commentrust-lang/mdBook

Add Bibliography with Bibtex-format support

What do you mean by external plugin?

Ah, I mean a preprocessor. A preprocessor can make arbitrary changes to the book content. (Plugins sort of loosely refer to either preprocessors or renderers.)

francisco-perez-sorrosal

comment created time in 5 hours

startedsoftprops/hyperlocal

started time in 5 hours

issue openednodejs/node

npm delete after any package install

<!-- Thank you for reporting an issue.

Version: 15.3.0 Platform: 64-bit (Windows) Subsystem:

often after install any package I get: Error: Cannot find module 'c\NodeJs\node_modules\npm\bin\npm-cli.js' and all the c\NodeJs\node_modules\npm is missing. Thus Node.js it becomes unusable and unnerving. How I can resolve it?

created time in 6 hours

issue openednodejs/node

It's still link to small-ciu with "vcbuild full-icu"

It's still link to small-ciu with "vcbuild full-icu" on windows: 1)download "icu4c-68_1-src.tgz" and unzip to /deps/icu 2)run "vcbuild full-icu dll" image

created time in 6 hours

PR opened rust-lang/mdBook

Add Katex preprocessor to 3rd Party Plugins

I developed a preprocessor rendering LaTex equations to HTML, which I think would be helpful for books with lots of equations. See #651 and #1105.

+1 -0

0 comment

1 changed file

pr created time in 6 hours

created repositoryFernandoCagale/calculator

created time in 6 hours

Pull request review commentnodejs/node

lib: add uncurried accessor properties to `primordials`

 function copyProps(src, dest) {   } } +function getNewKey(key) {+  return typeof key === 'symbol' ?+    `Symbol${key.description[7].toUpperCase()}${key.description.slice(8)}` :+    `${key[0].toUpperCase()}${key.slice(1)}`;+}+ function copyPropsRenamed(src, dest, prefix) {   for (const key of Reflect.ownKeys(src)) {-    if (typeof key === 'string') {-      Reflect.defineProperty(-        dest,-        `${prefix}${key[0].toUpperCase()}${key.slice(1)}`,-        Reflect.getOwnPropertyDescriptor(src, key));+    const newKey = getNewKey(key);+    const desc = Reflect.getOwnPropertyDescriptor(src, key);+    if ('get' in desc || 'set' in desc) {+      const { enumerable, get, set } = desc;+      if (get !== undefined) {

The code would be much simpler if we assume it is the case:

if ('get' in desc) {
  const { enumerable, get, set } = desc;
  Reflect.defineProperty(dest, `${prefix}Get${newKey}`, {
          value: uncurryThis(get),
          enumerable
  });
  if (set !== undefined) {
        Reflect.defineProperty(dest, `${prefix}Set${newKey}`, {
          value: uncurryThis(set),
          enumerable
        });
  }
}
ExE-Boss

comment created time in 6 hours

more