kellegous
Repos
90
Followers
106
Following
5

Events

golf

Created at 4 hours ago
create branch
kellegous create branch main
Created at 5 hours ago
create repository
kellegous create repository
Created at 5 hours ago

velocity = speed * impact

Created at 1 day ago
kellegous create branch main
Created at 1 week ago
create repository
kellegous create repository
Created at 1 week ago
kellegous create branch main
Created at 1 week ago
create repository
kellegous create repository
Created at 1 week ago

ugh

Created at 3 weeks ago

noodling on time

Created at 1 month ago

tinkering

Created at 1 month ago

.

Created at 1 month ago

noodling on frontend experiments

Created at 1 month ago

.

Created at 1 month ago

Upgrade all devDependencies in webpack build. (#439)

Created at 1 month ago
closed issue
webpack build fails with node 17+

Not sure the best way to integrate a fix into make ui but with node v18.2.0, I get the following error.

node:internal/crypto/hash:67
  this[kHandle] = new _Hash(algorithm, xofLen);
                  ^

Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:67:19)
    at Object.createHash (node:crypto:133:10)
    at module.exports (/Users/knorton/Documents/2022/09/hound/node_modules/webpack/lib/util/createHash.js:135:53)
    at NormalModule._initBuildHash (/Users/knorton/Documents/2022/09/hound/node_modules/webpack/lib/NormalModule.js:417:16)
    at handleParseError (/Users/knorton/Documents/2022/09/hound/node_modules/webpack/lib/NormalModule.js:471:10)
    at /Users/knorton/Documents/2022/09/hound/node_modules/webpack/lib/NormalModule.js:503:5
    at /Users/knorton/Documents/2022/09/hound/node_modules/webpack/lib/NormalModule.js:358:12
    at /Users/knorton/Documents/2022/09/hound/node_modules/loader-runner/lib/LoaderRunner.js:373:3
    at iterateNormalLoaders (/Users/knorton/Documents/2022/09/hound/node_modules/loader-runner/lib/LoaderRunner.js:214:10)
    at iterateNormalLoaders (/Users/knorton/Documents/2022/09/hound/node_modules/loader-runner/lib/LoaderRunner.js:221:10)
    at /Users/knorton/Documents/2022/09/hound/node_modules/loader-runner/lib/LoaderRunner.js:236:3
    at context.callback (/Users/knorton/Documents/2022/09/hound/node_modules/loader-runner/lib/LoaderRunner.js:111:13)
    at /Users/knorton/Documents/2022/09/hound/node_modules/babel-loader/lib/index.js:55:71
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

Node.js v18.2.0
make: *** [ui/bindata.go] Error 1

A work around for this issue can be found in the related webpack issue. https://github.com/webpack/webpack/issues/14532

For now, the best way to work around this is with

NODE_OPTIONS=--openssl-legacy-provider make ui
Created at 1 month ago

Upgrade all devDependencies in webpack build. (#439)

Created at 1 month ago
pull request closed
Upgrade all devDependencies in webpack build.

What kind of change does this PR introduce? (check at least one)

  • [ ] Bugfix
  • [ ] Feature
  • [ ] Code style update
  • [ ] Refactor
  • [x] Build-related changes
  • [ ] Other, please describe:

The PR fulfills these requirements:

  • [x] All tests are passing?
  • [ ] New/updated tests are included?
  • [x] If any static assets have been updated, has ui/bindata.go been regenerated?
  • [ ] Are there doc blocks for functions that I updated/created?

If adding a new feature, the PR's description includes:

  • [ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Fixes #438 - This issue was due to the fact that node 17 moved to openssl 3. webpack5 adjusted accordingly but webpack4 (which we were using) was not updated and will not be updated. This PR yolos us to webpack5.

Also fixes our dependabot alerts for vulnerabilities since it updates everything to the latest version.

I did some testing of the new build of the UI and everything seemed to be in good working order but if anyone else can give it a spin, I'd definitely appreciate the extra testing.

Created at 1 month ago

Convert Signal to es6 class. (#440)

Upgrade all devDependencies in webpack build.

Created at 1 month ago

Convert Signal to es6 class. (#440)

Created at 1 month ago

Convert Signal to es6 class. (#440)

Created at 1 month ago
pull request closed
Convert Signal to es6 class.

What kind of change does this PR introduce? (check at least one)

  • [ ] Bugfix
  • [ ] Feature
  • [ ] Code style update
  • [x] Refactor
  • [ ] Build-related changes
  • [ ] Other, please describe:

The PR fulfills these requirements:

  • [x] All tests are passing?
  • [x] New/updated tests are included?
  • [x] If any static assets have been updated, has ui/bindata.go been regenerated?
  • [x] Are there doc blocks for functions that I updated/created?

If adding a new feature, the PR's description includes:

  • [ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

I'd love to incrementally upgrade the ui code to leverage es6, including use of React.Component for the components. This also gives us the ability to split up the ui code into modules so that they are properly testable. I decided to start with a fairly self-contained class that is easily isolated and tested, Signal.

Created at 1 month ago

Convert Signal to es6 class.

Created at 1 month ago

Convert Signal to es6 class.

Created at 1 month ago