profile
viewpoint
Andy Edwards jedwards1211 @jcoreio Austin, TX coding hard, caving hard

google-map-react/google-map-react 4550

Google map library for react that allows rendering components as markers :tada:

gajus/flow-runtime 810

A runtime type system for JavaScript with full Flow compatibility.

howardroark/pollinate 208

Template your base files and generate new projects from Git(Hub).

jcoreio/crater 85

Meteor/Webpack/React SSR app skeleton that runs your app code outside of Meteor Isobuild

gregwebs/redux-side-effect 27

simple light-weight side effects for redux

jcoreio/react-router-drilldown 10

provides drilldown-style horizontal slide transitions between index and child routes

jedwards1211/async-child-process 6

Promise-based child process interface (you might want to use promisify-child-process instead!)

codemodsquad/jscodeshift-add-imports 2

add imports/requires if not already present with jscodeshift

codemodsquad/jscodeshift-build-import-list 1

build a list of all files and dependencies transitively imported by starting file(s)

codemodsquad/react-codemorphs 1

codemods for everyday work with React

pull request commentjcoreio/require-env

fix(deps): update dependency flow-bin to ^0.131.0

:tada: This PR is included in version 1.1.14 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

renovate[bot]

comment created time in 6 hours

release jcoreio/require-env

v1.1.14

released time in 6 hours

created tagjcoreio/require-env

tagv1.1.14

get value of an environment variable; throw if undefined or empty string

created time in 6 hours

startedpfusik/cito

started time in 13 days

push eventjcoreio/deploy-clarity-prod

Andy Edwards

commit sha 291929f5be788a1e5e560d72904a6ee1eb6239c5

upgrade @jcoreio/deploy-clarity to 6.0.2

view details

push time in 13 days

issue commentnodejs/tooling

the creeping scourge of tooling config files in project root directories

I get the impression half the reason Sindre Sorhus made XO is just so that he could have the linter defaults he wants without any extra configuration.

boneskull

comment created time in 13 days

issue commentnodejs/tooling

the creeping scourge of tooling config files in project root directories

@jkrems

could the number of those files be reduced somehow?

Personally I like putting config in package.json for tools that support it. Some people don't like the extra bytes winding up in the package though so it would be nice if npm had a built in way to exclude some keys in package.json from publishing.

Or maybe if you use something like https://github.com/romefrontend/rome, but it's hard to imagine Rome will be able to beat all these other specialty tools at their own game.

boneskull

comment created time in 13 days

push eventjcoreio/deploy-clarity-prod

Andy Edwards

commit sha b43b48acdca9fdf19e5e25cef3798aba8bb6fffb

upgrade @jcoreio/deploy-clarity to 6.0.1

view details

push time in 13 days

push eventjcoreio/deploy-clarity-prod

Andy Edwards

commit sha df534e4a1c8a4cdcc46875db2b67c7115d9dbdb1

fix typo in CI script

view details

push time in 13 days

push eventjcoreio/deploy-clarity-prod

Andy Edwards

commit sha 42fb9b5deffc3d8fb4dc0c4075e615a3f229c1e2

upgrade to @jcoreio/deploy-clarity 6.0.0

view details

push time in 13 days

push eventjcoreio/deploy-clarity-prod

Andy Edwards

commit sha 964e5e05bfced798187d8cefeaf5440e67d679cb

chore: add circleci config

view details

push time in 13 days

create barnchjcoreio/deploy-clarity-prod

branch : master

created branch time in 13 days

created repositoryjcoreio/deploy-clarity-prod

created time in 13 days

issue openedmokkabonna/inquirer-autocomplete-prompt

Pagination

Off the top of your head, do you know if it would be possible to implement pagination when the user scrolls down far enough in the search results? Or is it impossible to get notified by inquirer that that's happened?

created time in 14 days

pull request commentjcoreio/require-env

fix(deps): update dependency flow-bin to ^0.130.0

:tada: This PR is included in version 1.1.13 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

renovate[bot]

comment created time in 14 days

release jcoreio/require-env

v1.1.13

released time in 14 days

created tagjcoreio/require-env

tagv1.1.13

get value of an environment variable; throw if undefined or empty string

created time in 14 days

issue commentmicrosoft/vscode

Extensions using the "type" command (for ex. Vim) have poor performance due to being single-threaded with other extensions

@alexdima lately I've seen the builtin TS/JS extensions block Vim for several seconds when I'm in the middle of typing a function and it tries to rebuild the outline (I see an intermittent progress bar in the outline view while it's not taking keyboard input).

@tamuratak @jrieken @rebornix @joaomoreno I know you all want me to open a separate issues for other extensions' performance problems like this but please understand from my perspective that seems like it would perpetuate my suffering. I think the only hope of seeing a permanent solution where type commands run in a separate thread is to report other extensions' performance problems here.

DanTup

comment created time in 15 days

push eventjcoreio/poll

Andy Edwards

commit sha 247701fa632c9a00fd8562b27dada69078410d50

fix: failure to clear timeout immediately

view details

push time in 17 days

release jcoreio/poll

v2.1.1

released time in 17 days

created tagjcoreio/poll

tagv2.1.1

yet another promise-based poller

created time in 17 days

issue closedjcoreio/poll

The automated release is failing 🚨

:rotating_light: The automated release from the master branch failed. :rotating_light:

I recommend you give this issue a high priority, so other packages depending on you could benefit from your bug fixes and new features.

You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can resolve this 💪.

Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.

Once all the errors are resolved, semantic-release will release your package the next time you push a commit to the master branch. You can also manually restart the failed CI job that runs semantic-release.

If you are not sure how to resolve this, here is some links that can help you:

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.


Not running on Circle CI

Unfortunatly this error doesn't have any additionnal information. Feel free to kindly ask the author of the condition-circle plugin to add more helpful informations.


Good luck with your project ✨

Your semantic-release bot :package::rocket:

<!-- semantic-release:github -->

closed time in 17 days

jedwards1211

release jcoreio/poll

v2.1.0

released time in 17 days

created tagjcoreio/poll

tagv2.1.0

yet another promise-based poller

created time in 17 days

push eventjcoreio/poll

Andy Edwards

commit sha 56667b68c07eaf6f1945be25137910e2dbf66027

chore: fix CircleCI config

view details

push time in 17 days

push eventjcoreio/poll

Andy Edwards

commit sha ee21bc53d28138cc7078f08b25cf490c95004ff0

feat: add TypeScript defs

view details

push time in 17 days

issue commentyarnpkg/yarn

Yarn incorrectly started using `sha1` instead of `sha512`

@gabegorelick I'm confused when that would trigger the integrity mismatch issue then. I was getting a sha1/sha512 mismatch. Does that mean the hash in npm itself was upgrade from sha1 to sha512 but my local cache was old?

jonsadka

comment created time in 17 days

issue commentyarnpkg/yarn

"Incorrect integrity when fetching from the cache"

is there any way to upgrade a lockfile with the old sha-1 hashes to sha-256 without blowing away and recreating the entire lockfile?

xclidongbo

comment created time in 17 days

issue commentasvetliakov/vscode-neovim

Recommend to all users of this extension to upvote the perf issue in VSCode

@amanpalariya I think that's more a consequence of the fact that those editors didn't want to go to the trouble to make a plugin API and plugin marketplace. But a few IDEs do have built-in keys (It seems like a built-in Emacs option is more common, I imagine because Emacs is easier to emulate)

jedwards1211

comment created time in 19 days

issue commentjcoreio/react-view-slider

Input fields lose their focus on change

Thanks! Hope it goes well implementing the form, to me forms still feel more tedious to implement than they should be, at least with redux-form.

anuragsr

comment created time in 21 days

push eventjcoreio/docker-node-browsers-deploy

Andy Edwards

commit sha 2092048be39f692bf7ebc0566d827ed2db8842ff

fix: use node 10.18.1, fix pip issues

view details

push time in 21 days

pull request commentjedwards1211/smart-restart

feat: add TypeScript definitions

:tada: This PR is included in version 2.5.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

aleclarson

comment created time in 21 days

release jedwards1211/smart-restart

v2.5.0

released time in 21 days

created tagjedwards1211/smart-restart

tagv2.5.0

Like nodemon but only watches require()d files. Like piping but easy to use with node-inspector.

created time in 21 days

push eventjedwards1211/smart-restart

Andy Edwards

commit sha 833b29f537664190afed153dd1a633493f1cc376

docs(README.md): fix status badge

view details

push time in 21 days

release jedwards1211/smart-restart

v2.4.0

released time in 21 days

created tagjedwards1211/smart-restart

tagv2.4.0

Like nodemon but only watches require()d files. Like piping but easy to use with node-inspector.

created time in 21 days

delete tag jedwards1211/smart-restart

delete tag : v2.4.0

delete time in 21 days

release jedwards1211/smart-restart

v2.3.0

released time in 21 days

created tagjedwards1211/smart-restart

tagv2.4.0

Like nodemon but only watches require()d files. Like piping but easy to use with node-inspector.

created time in 21 days

PR merged jedwards1211/smart-restart

feat: add TypeScript definitions

Note: I wanted to export the LaunchOptions and LaunchProcess types, but export = syntax does not allow it. But no biggie, I probably won't need them. 😄

+29 -0

4 comments

1 changed file

aleclarson

pr closed time in 21 days

push eventjedwards1211/smart-restart

Alec Larson

commit sha a1d4359cded87578d22286b61b3cdc259733002c

feat: add TypeScript definitions

view details

Andy Edwards

commit sha 4632337ac7a0078864e111651bcdcd5b9486deaf

feat: add default export, TypeScript and Flow type defs

view details

Andy Edwards

commit sha cef29bf72dde37f448c618ee318ece761e7a8a68

Merge branch 'aleclarson-feat/dts'

view details

push time in 21 days

pull request commentjedwards1211/smart-restart

feat: add TypeScript definitions

that's a good point though. I could go ahead and do that too

aleclarson

comment created time in 21 days

pull request commentjedwards1211/smart-restart

feat: add TypeScript definitions

well, I'm going to merge this with some alterations, like also flow type defs and making sure the CI build checks the defs

aleclarson

comment created time in 21 days

pull request commentjedwards1211/smart-restart

feat: add TypeScript definitions

Oh, hmmm. That's too bad, Flow is actually more flexible in this case. I guess turning this into a Dual CommonJS/ES Module Package would be better for the TS types?

aleclarson

comment created time in 21 days

issue commentjedwards1211/smart-restart

Thanks

Huh, for some reason I didn't find that last time I looked. I should check it out!

aleclarson

comment created time in 21 days

issue commentjedwards1211/smart-restart

Thanks

You're welcome! And I have to thank @mdlawson for coming up with the general approach in his package piping. At times I've wanted to make a Webpack-style hot reloading API for node, because one of my app projects is getting pretty bloated and slow to restart. I've never gotten around to it though... I think it would be possible to mimic module.hot almost exactly though.

aleclarson

comment created time in 21 days

pull request commentmnajdova/material-ui

Utility classes

Seems like we would just need to make a PR to that html-css-class-completion extension that lets us configure additional directories inside node_modules to scan for CSS classes

mnajdova

comment created time in 21 days

release jcoreio/mindless-route53

v1.2.4

released time in 21 days

created tagjcoreio/mindless-route53

tagv1.2.4

low-mental-load API/CLI for AWS Route 53

created time in 21 days

push eventjcoreio/mindless-route53

Andy Edwards

commit sha bade69eaf14d632e57fd58100d818659263b47b5

fix: move @types/deep-equal to devDependencies

view details

push time in 21 days

release jcoreio/mindless-route53

v1.2.3

released time in 21 days

created tagjcoreio/mindless-route53

tagv1.2.3

low-mental-load API/CLI for AWS Route 53

created time in 21 days

push eventjcoreio/mindless-route53

Andy Edwards

commit sha a68cb4bdb59e38bd8ced4f819c607a1c84eceb7d

fix: move deep-equal to dependencies

view details

push time in 21 days

release jcoreio/mindless-route53

v1.2.2

released time in 21 days

created tagjcoreio/mindless-route53

tagv1.2.2

low-mental-load API/CLI for AWS Route 53

created time in 21 days

push eventjcoreio/mindless-route53

Andy Edwards

commit sha 8680d398b296d99535752d1f36030c0424cb68df

fix: flow and eslint errors

view details

push time in 21 days

push eventjcoreio/mindless-route53

Andy Edwards

commit sha 11cf8e24a62b4e727e56bc0d73a9a43d4b335409

test: fix broken tests

view details

push time in 21 days

push eventjcoreio/mindless-route53

Andy Edwards

commit sha 56d704786bd8cc7ffa550c4433d0c406bf24dce7

fix: don't upsert if identical record exists, to save time

view details

push time in 21 days

issue openedaws/aws-sdk-js

Route53.listResourceRecordSets requires MaxItems to be a string -- shouldn't it be a number?

Confirm by changing [ ] to [x] below to ensure that it's a bug:

Title says it all.

created time in 21 days

release jcoreio/typescript-validators

v1.0.1

released time in 21 days

created tagjcoreio/typescript-validators

tagv1.0.1

API input validators with user-friendly error output and TypeScript to ensure you don't miss any properties

created time in 21 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 94410f83b3979d6711c9e0b5357773df8540db85

fix(instanceOf): fix output type

view details

push time in 21 days

issue commentjcoreio/react-view-slider

Input fields lose their focus on change

Here's an example of what I mean about different function instances each time:

function App() {
    const EmailForm = () => {}
    return EmailForm
}
const emailFormA = App()
const emailFormB = App()
console.log(emailFormA === emailFormB) // false

Unfortunately, even though the code in EmailForm hasn't changed, JS creates a new function instance for it. If JS returned the same function instance every time, on the second render EmailForm would be bound to the old value of formData, not the new value.

When you render <EmailForm ... />, if EmailForm is !== what it was on the last render, React will remount it (remove all the DOM elements from the previous render and replace them with new DOM elements). This remounting was what caused the input to lose focus.

anuragsr

comment created time in 21 days

issue closedjcoreio/react-view-slider

Input fields lose their focus on change

I'm using the below code in my App.js:

 <SimpleViewSlider 
       className="outer"
       viewStyle={{ height: "100%" }}
       viewportStyle={{ height: "100%" }}
       innerViewWrapperStyle={{ height: "100%" }}
       >
    <div className="page" key={page}>
         This is view {page}
         {content}
    </div>
 </SimpleViewSlider>

In {content}, I have an input text field <input type="text" name="email" value={email} onChange={setForm}/>

The issue is, that <div className="page"> is rerendered on change of page every time and that means the inputs lose focus as they are changed.

I'm building a multistep form, pretty much followed the code here, without Material UI. But there doesn't seem to be any input binding to state in the code (useState)

How can I ensure the input fields stay in focus?

closed time in 21 days

anuragsr

issue commentjcoreio/react-view-slider

Input fields lose their focus on change

Thanks for the code sandbox, now I can see the problem: EmailForm and EmailForm2 are remounting on every render because they're set to a new function instance every time App is rendered. If you use the same function instance every time the input won't lose focus. So I'd recommend moving those component functions outside of App. Here's an example of that, you can see it solves the focus problem: https://codesandbox.io/s/youthful-shannon-7fsto?file=/src/App.js

Just for info, if you were to implement a state in your demo so that at the last click, the user has all the form values in an object, what would be the approach?

Well, I was just trying to make the example simple. I don't actually use local state for form values in my projects, I use redux-form. But even redux-form is kind of an old project and its author made a more modern library for form state called react-final-form 😅.

If you want to use either of those libraries they have examples of how to implement this type of form:

react-final-form multi-step form: https://final-form.org/docs/react-final-form/examples/wizard redux-form multi-step form: https://redux-form.com/8.3.0/examples/wizard/

anuragsr

comment created time in 21 days

push eventjcoreio/cloudformation-tools

Andy Edwards

commit sha 13d112477cd0c264390dac70e4e33499c7513fea

fix: add vpc TypeScript defs

view details

push time in 22 days

release jcoreio/mindless-route53

v1.2.1

released time in 22 days

created tagjcoreio/mindless-route53

tagv1.2.1

low-mental-load API/CLI for AWS Route 53

created time in 22 days

push eventjcoreio/mindless-route53

Andy Edwards

commit sha 9ca9060a45d154550a69c68616c67e2fb4bc1059

fix: add TypeScript defs for upsertPublicAndPrivateRecords

view details

push time in 22 days

push eventjcoreio/cloudformation-tools

Andy Edwards

commit sha ba0ba7e27ee1750ac8b4c4f32b98b9d5ae77d229

feat: add TypeScript defs for security groups stuff

view details

push time in 22 days

issue commentjcoreio/react-view-slider

Input fields lose their focus on change

You should pass the global state and callback down to the input, but that isn't actually what's causing the input to lose focus. Are you saying that for example page 1 and page 2 both have email fields and you want the email field on page 2 to be focused after you transition to it from page 1?

anuragsr

comment created time in 22 days

issue commentjcoreio/react-view-slider

Input fields lose their focus on change

useAutofocusRef is what focuses the inputs in that example. If you use it in a content compnent like

import * as React from 'react'
import { useAutofocusRef } from 'react-transition-context'

const Page = ({ page }) => {
  // this is just an example, you could also manage field values with redux or a form state library
  const [email, setEmail] = React.useState("")
  const handleChange = React.useCallback((e) => setEmail(e.target.value), [
    setEmail,
  ])
  const autofocusRef = useAutofocusRef()

  return (
    <div className="page">
      This is view {page}
      <input
        type="text"
        name="email"
        value={email}
        onChange={handleChange}
        ref={autoocusRef}
      />
    </div>
  )
}

and then later render it like this, it should focus the input.

<SimpleViewSlider
  className="outer"
  viewStyle={{ height: "100%" }}
  viewportStyle={{ height: "100%" }}
  innerViewWrapperStyle={{ height: "100%" }}
>
  <Page key={page} page={page} />
</SimpleViewSlider>
anuragsr

comment created time in 22 days

issue commentsindresorhus/p-limit

Should clearQueue() reject the pending promises?

@RichardBronosky maybe you have an old version installed? Looks like clearQueue was only added 3 months ago.

jedwards1211

comment created time in 23 days

release jcoreio/mindless-route53-cloudformation

v1.2.0

released time in 24 days

created tagjcoreio/mindless-route53-cloudformation

tagv1.2.0

Intelligently create DNS entries an EC2 Instance or load balancer output from a CloudFormation stack

created time in 24 days

push eventjcoreio/mindless-route53-cloudformation

Andy Edwards

commit sha c9777bc6d1dd154d17472acfc9f0c11221471ac4

feat: add TypeScript types

view details

push time in 24 days

release jcoreio/mindless-route53

v1.2.0

released time in 24 days

created tagjcoreio/mindless-route53

tagv1.2.0

low-mental-load API/CLI for AWS Route 53

created time in 24 days

push eventjcoreio/mindless-route53

Andy Edwards

commit sha 372100ffcc403d1b19731b60af0cead679b81b7d

feat: add TypeScript defs

view details

push time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha cd3227889c4161e839f0d5ce158921a581344017

chore: re-enable coverage badge

view details

push time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 302588f8a24d4453fef48eea49301bc7e1c7f766

chore: format README

view details

push time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 89d911413e03fd4f7e54506322532e6bf2aa23a8

chore: upgrade codecov client

view details

push time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 71b7e2d3bd5f058fc12f5801c75989892bcb5f3d

docs(README.md): hide coverage until codecov is working

view details

push time in 24 days

push eventjcoreio/aws-utils

Andy Edwards

commit sha 4d52755227e3bfcb7e6b67a2382e9b2a66e02c20

feat(aws-sdk-async-iterables/ec2): add all paged describe methods affects: @jcoreio/aws-sdk-async-iterables

view details

push time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha d00905367e69da770e3c1518a0626084e197ccbd

docs(README.md): add note about readonly props

view details

push time in 24 days

release jcoreio/typescript-validators

v1.0.0

released time in 24 days

created tagjcoreio/typescript-validators

tagv1.0.0

API input validators with user-friendly error output and TypeScript to ensure you don't miss any properties

created time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 359cfa3d7f9ddb080b0de658955d96ff9fd4f628

docs(README.md): fix capitalization

view details

push time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 6fa30c717610357eb5edfce30c87da1d33eb78db

docs(README.md): fix links

view details

push time in 24 days

startedshannonmoeller/find-config

started time in 24 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha e076583924536f897a553ddbea0adf3734ed36ca

docs(README.md): fix typo

view details

push time in 25 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 01568776c7e81a627f673e9faf586b25faaaea2c

fix: eslint errors

view details

push time in 25 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha efa6fe240c16090abb7d286d32a01200ec11024d

fix: records, test intersections and unions

view details

push time in 25 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha be40ce2f260a75dcce7a3959ac64df360ff24079

fix(symbol): fix toString() and add tests

view details

Andy Edwards

commit sha 26fc46cfe2d65fdfff4ffb4040e1815e713bd546

fix: object property validation and more tests

view details

push time in 25 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 21eeaaf58726d6a7454c5e14dbbde614faaf57de

docs(README.md): fix typo

view details

push time in 25 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 8aa87305934edee15ad80a57147ca8e4b3d9a908

feat: add alias and ref for recursive types `t.constrain` has been replaced with `t.alias`. See the README for more details. BREAKING CHANGE: remove t.constrain (replaced by t.alias)

view details

push time in 25 days

push eventjcoreio/typescript-validators

Andy Edwards

commit sha 0b665b84952524b647be7b6ac180613a3c67fa3c

fix: remove unused API

view details

push time in 25 days

more