profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/kibertoad/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.
Igor Savin kibertoad Vilnius Principal engineer at Normalis Tech. Lead maintainer of @knex

fastify/fastify-request-context 48

Request-scoped storage support, based on Asynchronous Local Storage (with fallback to cls-hooked)

kibertoad/asynchronous-local-storage 21

Asynchronous local storage implementation based on Node.js ALS with fallback to cls-hooked for older Node.js versions

fastify/fastify-awilix 19

Dependency injection support for fastify

fastify/fastify-funky 15

Make fastify functional! Plugin, adding support for fastify routes returning functional structures, such as Either, Task or plain parameterless function.

fastify/fastify-schedule 12

Fastify plugin for scheduling periodic jobs.

7upcat/mobx-event-bus 7

A super tiny event bus build upon Mobx allows publish-subscribe-style communication between components.

caryoscelus/hired-guns 1

a game in development

caryoscelus/psychobear 1

The bears be here

JakeGinnivan/knex 1

A query builder for PostgreSQL, MySQL and SQLite3, designed to be flexible, portable, and fun to use.

kibertoad/algorithmicon 1

Code kata exercises and useful code snippets (monoku)

pull request commentjorgebucaran/colorette

Add support for CI envs in colour support check

@jorgebucaran It makes reading logs in CI console easier, and also this is a default behaviour for support-colours, so I would say that this is what the majority of users are actually expecting.

kibertoad

comment created time in 16 minutes

delete branch kibertoad/colorette

delete branch : feat/ci-support

delete time in 17 minutes

pull request commentpinojs/pino

Fix typo

Thanks!

vladshcherbin

comment created time in 10 hours

push eventpinojs/pino

Vlad Shcherbin

commit sha 1c98f752522c9fbf6e141d1cc5cd13f76ae03012

Fix typo (#1084)

view details

push time in 10 hours

PR merged pinojs/pino

Fix typo
+2 -2

0 comment

1 changed file

vladshcherbin

pr closed time in 10 hours

PullRequestReviewEvent

Pull request review commentjorgebucaran/colorette

Add support for CI envs in colour support check

-let enabled =-  !("NO_COLOR" in process.env) &&-  ("FORCE_COLOR" in process.env ||-    process.platform === "win32" ||-    (process.stdout != null &&-      process.stdout.isTTY &&-      process.env.TERM &&-      process.env.TERM !== "dumb"))+const {env} = process;++const isDisabled = "NO_COLOR" in env+const isForced = "FORCE_COLOR" in env+const isWindows = process.platform === "win32"++const isCompatibleTerminal = (process.stdout != null &&+    process.stdout.isTTY &&+    env.TERM &&+    env.TERM !== "dumb")++const isCI = ('CI' in env && ['CIRCLECI', 'GITLAB_CI', 'GITHUB_ACTIONS'].some(sign => sign in env))++let enabled = (!isDisabled) &&+  (isForced || isWindows || isCompatibleTerminal || isCI)

Done!

kibertoad

comment created time in 2 days

PullRequestReviewEvent

push eventkibertoad/colorette

Igor Savin

commit sha fab0aad97db56249aec98f74ff5d8c7fcdd8343f

Address comments

view details

push time in 2 days

PR opened jorgebucaran/colorette

Add support for CI envs in colour support check

fixes #61

+17 -9

0 comment

2 changed files

pr created time in 3 days

create barnchkibertoad/colorette

branch : feat/ci-support

created branch time in 3 days

fork kibertoad/colorette

Easily set the color and style of text in the terminal.

fork in 3 days

issue commentjorgebucaran/colorette

Support for CI environments is lacking

@jorgebucaran Sure, although I would also include GitLab there, lots of major players are using it.

kibertoad

comment created time in 4 days

issue commentjorgebucaran/colorette

Support for CI environments is lacking

@jorgebucaran Would you accept a PR for this?

kibertoad

comment created time in 4 days

issue commentpinojs/pino-pretty

Switch from chalk to colorette

@mcollina Generally I would prefer to see this functionality upstream, the only reason why it might be preferable to have it in pino-pretty is being able to lock to slightly older version to avoid stumbling upon https://github.com/jorgebucaran/colorette/issues/48. Do we officially support Node 13?

kibertoad

comment created time in 4 days

Pull request review commentknex/knex

fix typing deferrable and withkeyName should not be in ColumnBuilder

 const main = async () => {   expectType<Pick<User, 'id'> | undefined>(     await knexInstance.first('id').from('users_composite')   );++  //These tests simply check if type work by showing that it does not throw syntax error++  knexInstance.schema.createTable('testTable',(table) => {+    table.foreign('fkey_three').references('non_exist.id').withKeyName('non_for1').deferrable('deferred');+    table.foreign('fkey_threee').references('non_exist.id').deferrable('deferred').withKeyName('non_for2');+    table.integer('num').references('non_exist.id').deferrable('immediate').withKeyName('non_for3');+    table.integer('num').references('non_exist.id').withKeyName('non_for4').deferrable('deferred').onDelete('CASCADE');+    table.integer('num').references('non_exist.id').withKeyName('non_for5').deferrable('deferred').onDelete('CASCADE');+    table.integer('num').references('id').inTable('non_exist').withKeyName('non_for6').deferrable('deferred').onDelete('CASCADE');+    table.integer('num').references('id').withKeyName('non_for7').deferrable('deferred').inTable('non_exist').onDelete('CASCADE');+  })

here is good!

zeotuan

comment created time in 4 days

PullRequestReviewEvent

Pull request review commentknex/knex

fix typing deferrable and withkeyName should not be in ColumnBuilder

 export declare namespace Knex {     index(indexName?: string, indexType?: string): ColumnBuilder;   } -  interface ReferencingColumnBuilder extends ColumnBuilder {-    inTable(tableName: string): ColumnBuilder;+  interface ReferencingColumnBuilder extends ColumnBuilder{+    inTable(tableName: string): ReferencingColumnBuilder;+    deferrable(type: deferrableType): ReferencingColumnBuilder;+    withKeyName(keyName: string): ReferencingColumnBuilder;

if it wouldn't be too hard!

zeotuan

comment created time in 4 days

PullRequestReviewEvent

push eventkibertoad/dealing-with-complexity

Igor Savin

commit sha 22989c900d04bb67474eccd394f534f5db0ec581

Initial implementation

view details

push time in 4 days

create barnchkibertoad/dealing-with-complexity

branch : main

created branch time in 4 days

created repositorykibertoad/dealing-with-complexity

Source code for the "Dealing with Complexity" workshop

created time in 4 days

issue commentpinojs/pino-pretty

Switch from chalk to colorette

Currently blocked by https://github.com/jorgebucaran/colorette/issues/61

kibertoad

comment created time in 4 days

issue openedjorgebucaran/colorette

Support for CI environments is lacking

Current implementation doesn't reliably determine colour support in CI environments, resulting e. g. in CI for https://github.com/pinojs/pino-pretty/tree/feat/colorette to break.

I think these lines from supports-colour should be included:

	if ('CI' in env) {
		if (['TRAVIS', 'CIRCLECI', 'APPVEYOR', 'GITLAB_CI', 'GITHUB_ACTIONS', 'BUILDKITE', 'DRONE'].some(sign => sign in env) || env.CI_NAME === 'codeship') {
			return 1;
		}

		return min;
	}

created time in 4 days

pull request commentforeversd/forever

Remove dependency on timespan

@kanish-111 Matteo Collina doesn't recommend pm2 due to it monkey-patching lots of Node.js internals, and I trust his authority on this.

kibertoad

comment created time in 4 days

push eventpinojs/pino-pretty

Igor Savin

commit sha 6e661e93fca61135fc672f1801e0eaf552089c55

Try to fix tests

view details

push time in 4 days

pull request commentforeversd/forever

Remove dependency on timespan

@kanish-111 All of them are pretty lighthweight. Nodemon has the best support these days.

kibertoad

comment created time in 4 days

create barnchpinojs/pino-pretty

branch : feat/colorette

created branch time in 4 days

pull request commentforeversd/forever

Remove dependency on timespan

@kanish-111 Why not use nodemon?

kibertoad

comment created time in 4 days

pull request commentknex/knex

fix typing deferrable and withkeyName should not be in ColumnBuilder

@zeotuan I would simply invoke a builder chain in a test that used to throw a type error in the past. Since from my understanding this particular PR adds support for this method for more different builder cases, using several different chains would be the best. I don't think there will be any actual assertions in new test code, just some code that wouldn't have compiled in the past.

zeotuan

comment created time in 4 days