profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/MrArnoldPalmer/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.
Mitchell Valine MrArnoldPalmer AWS Los Angeles, CA Opinions are my own

construct-catalog/catalog 109

Catalog for aggregating CDK Constructs

MrArnoldPalmer/re-frame-git 2

ClojureScript project drawing graphs based on git repository data

MrArnoldPalmer/jsii 1

Use javascript modules naturally from every programming language

MrArnoldPalmer/kompose 1

JS Composition Tools/Utils

MrArnoldPalmer/angular-phonecat-re-frame 0

Port of the angular phonecat tutorial to re-frame

MrArnoldPalmer/aws-cdk 0

The AWS Cloud Development Kit is a framework for defining cloud infrastructure in code

MrArnoldPalmer/aws-cdk-examples 0

Example projects using the AWS CDK

MrArnoldPalmer/aws-delivlib 0

setup and manage continuous delivery pipelines for code libraries in multiple languages

MrArnoldPalmer/aws-gameservers 0

cdk constructs for deploying various game servers to aws

PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent

issue commentaws/aws-cdk

(aws-cdk/aws-amplify): Add a sourceCodeProvider for S3 or Zipped Asset

Oh that's a good callout. We may want to experiment with object versioning on the bucket where we store the assets and perhaps we could revert a version on a rollback.

samkio

comment created time in 5 days

PullRequestReviewEvent

issue commentcdklabs/construct-hub-webapp

Very slow page loading times

So we have a number of plans related to performance.

  1. Switching the output of the backend from .md to json files. There is significant client side CPU time rendering large api reference documents in markdown using ReactMarkdown. In experiments, passing a json document and iterating through that ourselves performs better and gives the client more control over the layout of the api reference without requiring backfills.

  2. Migrating to NextJS SSR/SSG. This would cut down on data sent over the wire and on client side computation. What remains is to figure out the best way to run this within construct-hub using some serverless architecture. Amplify is an option as well as existing serverless-nextjs constructs.

We aren't really dedicated to either of these right now but the plan is to implement them within the next couple months.

gshpychka

comment created time in 5 days

push eventcdklabs/construct-hub

Christopher Rybicki

commit sha 0940cd8a9b3fa6b11438e39dd0601aaba63d8a38

chore: upgrade dependencies (#328)

view details

Mitchell Valine

commit sha 0ef92a222ed5d3d7c2f0de44d944af558362ec9f

Merge branch 'main' of github.com:cdklabs/construct-hub into feat/webapp-runtime-configuration

view details

push time in 5 days

PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentcdklabs/construct-hub-webapp

chore: cleanup package filter logic

 export const LANGUAGE_NAME_MAP: { readonly [key in Language]: string } = { export const TEMP_SUPPORTED_LANGUAGES: ReadonlySet<Language> = new Set([   Language.Python,   Language.TypeScript,+  Language.Java,

We have a separate PR only for adding java support when rdy here https://github.com/cdklabs/construct-hub-webapp/pull/384/files

gabewomble

comment created time in 5 days

PullRequestReviewEvent

Pull request review commentcdklabs/construct-hub-webapp

chore: cleanup package filter logic

 export interface UseCatalogResultsOptions {  */ export const useCatalogResults = ({   limit,-  offset,-  query,-  language,+  offset = 0,+  query = "",+  language = null, }: UseCatalogResultsOptions) => {   const { data, loading, error } = useCatalog();    const results = useMemo(() => {     if (loading || error || !data?.packages) return []; -    let filtered = data.packages;+    const filtered = data.packages.filter((item) => {+      // Filter out items which do not support the current language+      if (language && language !== Language.TypeScript) {+        if (!Object.keys(item.languages).includes(language)) {+          return false;+        }+      }++      // Filter items which do not match the current query+      if (query.length) {+        const itemStr = JSON.stringify(item).toLowerCase();+        const queryStr = query.toLowerCase(); -    if (query.length) {-      filtered = filtered.filter((item) =>-        JSON.stringify(item).toLowerCase().includes(query.toLowerCase())-      );-    }+        if (!itemStr.includes(queryStr)) {+          return false;+        }+      } -    // Omit TS from filters b/c it is always available-    if (language && language !== Language.TypeScript) {-      filtered = filtered.filter((item) =>-        Object.keys(item.languages).includes(language)-      );-    }+      return true;+    }); -    return filtered.sort((p1, p2) => {+    // Next sort by most recently updated. (The sort should eventually become a parameter)+    return [...filtered].sort((p1, p2) => {

what is the need to copy the array over before sorting?

gabewomble

comment created time in 5 days

PullRequestReviewEvent
PullRequestReviewEvent

PR opened cdklabs/construct-hub-webapp

Reviewers
fix: revert .projenrc to before build time config

Another attempt at fixing bundle content to include the build file. Verified that running yarn build && yarn package results in a tarball with the expected build directory.

Fixes #

+17 -4

0 comment

4 changed files

pr created time in 6 days

create barnchcdklabs/construct-hub-webapp

branch : fix/release-bundle

created branch time in 6 days

push eventcdklabs/construct-hub

Mitchell Valine

commit sha 145cfc637ce838714a689b64dbb14672b7c22826

chore: upgrade depdendencies

view details

push time in 6 days

PR opened cdklabs/construct-hub-webapp

Reviewers
fix: release bundle missing build dir

Fixes #

+7 -38

0 comment

5 changed files

pr created time in 6 days

create barnchcdklabs/construct-hub-webapp

branch : fix/release-bundle

created branch time in 6 days

push eventcdklabs/construct-hub

Mitchell Valine

commit sha 97d8311fdb31b03c364cda9a5f9abab36e6b6e9e

chore: upgrade depdendencies

view details

push time in 6 days

PR opened cdklabs/construct-hub

chore: upgrade depdendencies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

+1981 -1951

0 comment

13 changed files

pr created time in 6 days

create barnchcdklabs/construct-hub

branch : chore/upgrade-deps

created branch time in 6 days

PullRequestReviewEvent

push eventcdklabs/construct-hub

Mitchell Valine

commit sha 9a4ad5b13e0d188a75687f3e287a2b05599c6fac

PR feedback

view details

push time in 6 days

push eventcdklabs/construct-hub

Mitchell Valine

commit sha 39215e192390ac29ffda56625027a5bc28269407

add test

view details

Mitchell Valine

commit sha 98abe51ebce75bda810ed49a488e9fcb0f601ad2

Merge branch 'feat/webapp-runtime-configuration' of github.com:cdklabs/construct-hub into feat/webapp-runtime-configuration

view details

Mitchell Valine

commit sha 899d00cea32803052a5487c5c5e122c32f8b796d

update snapshots

view details

push time in 6 days

push eventcdklabs/construct-hub

Christopher Rybicki

commit sha 5c8990cbd23a37ebcf748819c2bc967b77bf59f2

feat: Java API reference (#320) Adds Java API reference generation to the backend. I tested this by deploying to my dev stack, and re-triggering the doc generating step function for cdk8s, and afterwards the Java docs were available successfully at <xxx.cloudfront.net>/data/cdk8s/v1.0.0-beta.41/docs-java.md. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

view details

Mitchell Valine

commit sha e9be9ab005249c9bd1a4a932fe0b977057a6989d

fix: broken release workflow snapshot tests (#323) The release workflow is currently broken because when running `build` after running `bump`, the snapshot content has changed due to the version number being included in links to the runbook. This changes the url to always reference the latest version. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

view details

Mitchell Valine

commit sha 13083430f27c00ad0f75114497a70ed4a0636d1b

fix: broken error handling in npmjs follower (#324) A thrown error in the `httpGet` function within the npm js follower lambda causes the entire function to fail instead of just logging the individual package that it is unable to access. Changed the error handling block to invoke the catch branch of the promise returned. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

view details

Mitchell Valine

commit sha 4038de498a5f957cceeb2ea9de350271b4eaa84c

Merge branch 'main' of github.com:cdklabs/construct-hub into feat/webapp-runtime-configuration

view details

push time in 6 days

issue commentaws/aws-cdk

(aws-cdk/aws-amplify): Add a sourceCodeProvider for S3 or Zipped Asset

That is great! We definitely should make a plan to get this into @aws-cdk/aws-amplify. I'm not sure if we have precedence for generating signed URL's elsewhere in custom resources but it doesn't see too hard to accomplish based on the PoC. It probably is also worth thinking about this from a security perspective, passing a presigned URL in the body of an encrypted request seems relatively safe but I wonder if there is something I'm not thinking about?

samkio

comment created time in 6 days

push eventcdklabs/construct-hub

Mitchell Valine

commit sha e9be9ab005249c9bd1a4a932fe0b977057a6989d

fix: broken release workflow snapshot tests (#323) The release workflow is currently broken because when running `build` after running `bump`, the snapshot content has changed due to the version number being included in links to the runbook. This changes the url to always reference the latest version. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

view details

Mitchell Valine

commit sha 3c2a7a0fd110b774851a2d3c9bbefc9d096c6b79

Merge branch 'main' into fix/npm-ingestion-error-handling

view details

push time in 8 days

PR opened cdklabs/construct-hub

fix: broken error handling in npmjs follower

A thrown error in the httpGet function within the npm js follower lambda causes the entire function to fail instead of just logging the individual package that it is unable to access.

Changed the error handling block to invoke the catch branch of the promise returned.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

+149 -149

0 comment

3 changed files

pr created time in 8 days

create barnchcdklabs/construct-hub

branch : fix/npm-ingestion-error-handling

created branch time in 8 days