hyperx icon indicating copy to clipboard operation
hyperx copied to clipboard

Can I become a maintainer on this project?

Open mreinstein opened this issue 3 years ago • 1 comments

I'd like to do a major release for hyperx:

  • several fixes for open parsing bugs.
    • #41
    • #82
  • optionally consider adopting es6-isms (let, const, etc.)
  • optionally consider es modules since all LTS node versions and most browsers have support now

@substack @goto-bus-stop is this something you'd be interested in? This module underpins a lot of stuff for projects I work on and being able to fix some of the bit rot would be nice, so I don't have to maintain my own fork, and others could benefit too.

mreinstein avatar Jan 01 '22 16:01 mreinstein

I'd like to consider refactoring parse(str) to be an explicit state machine. Rather than having very long if/else if chains we could have a separate function for each state, something like this:

function open (c, m) { … }

function attr_key (c, m) { … }

var lookup = {
  OPEN: open,
  ATTR_KEY: attr_key,
  ⋮
}

function parse (str) {
    var i, machine = {
         state: TEXT,
         res: [ ],
         reg: ''
    }

    for (i=0; i < str.length; i++)
      lookup[machine.state](str[i], machine)

    return machine.res
}

mreinstein avatar Jan 02 '22 18:01 mreinstein

Just ran into the #41 issue .. @goto-bus-stop @mreinstein I'd also like to volunteer to help maintain.

rook2pawn avatar Apr 29 '23 02:04 rook2pawn

I would love someone to take over ownership of this project but sadly it is a mausoleum.

mreinstein avatar Apr 29 '23 04:04 mreinstein

This fork has my latest https://github.com/mreinstein/hyperx

mreinstein avatar Apr 29 '23 04:04 mreinstein

@goto-bus-stop @substack I'm still keen to maintain this project. Any chance of letting me do this? I use hyperx in my work and personal projects and I'm really not crazy about maintaining these personal forks if I can avoid it.

mreinstein avatar Oct 28 '23 20:10 mreinstein

@mreinstein thanks for the interest 😸 I sent an invitation to the GitHub org. I can't invite to npm on my phone I think but I can do that later.

goto-bus-stop avatar Oct 29 '23 11:10 goto-bus-stop

Nice! My username on npm is mreinstein.

I'm assuming you're too busy with other things to maintain this, but if you'd like I'm happy to include you on PRs/activity stuff. My immediate plans are to take the changes in my fork and get them published into choojs/hyperx

thanks again!

-Michael

On Sun, Oct 29, 2023 at 4:30 AM Renée @.***> wrote:

@mreinstein https://github.com/mreinstein thanks for the interest 😸 I sent an invitation to the GitHub org. I can't invite to npm on my phone I think but I can do that later.

— Reply to this email directly, view it on GitHub https://github.com/choojs/hyperx/issues/83#issuecomment-1784075513, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFPJ4YW3TFVCST7BZKDRVTYBYV5VAVCNFSM5LCUS3U2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCNZYGQYDONJVGEZQ . You are receiving this because you were mentioned.Message ID: @.***>

mreinstein avatar Oct 29 '23 15:10 mreinstein

Yeah I can't realistically commit to doing reviews. feel free to request them especially if you do need input on things, but I'm not actively using these projects at the moment so I think my opinions on direction are not the most important, and I may not find time/motivation to do much anyways 🙂

goto-bus-stop avatar Oct 29 '23 17:10 goto-bus-stop