choojs / hyperx

🏷 - tagged template string virtual dom builder
BSD 2-Clause "Simplified" License
1.01k stars 48 forks source link

Can I become a maintainer on this project? #83

Closed mreinstein closed 1 year ago

mreinstein commented 2 years ago

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

@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 commented 2 years ago

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
}
rook2pawn commented 1 year ago

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

mreinstein commented 1 year ago

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

mreinstein commented 1 year ago

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

mreinstein commented 1 year ago

@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.

goto-bus-stop commented 1 year ago

@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.

mreinstein commented 1 year ago

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: @.***>

goto-bus-stop commented 1 year ago

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 🙂