Update node.js.yml - Githubissues

RoseTheFlower / MetroSteam

Metro skin for Steam. Reborn.

184 stars 8 forks source link

Update node.js.yml #31

Closed shdwmtr closed 6 months ago

shdwmtr commented 6 months ago

Change Notes:

make workflow dispatch on manual call
update to use git module from organization instead of local

RoseTheFlower commented 6 months ago

Thanks. I delete the branch and do over, since we're after a one-time change and the whole commit history isn't needed. I'll just incorporate the repo change by hand, while the push approach probably doesn't matter either way.

RoseTheFlower commented 6 months ago

It also breaks things, since the letter case change results in the new directory being ClassMapper, not classmapper.

shdwmtr commented 6 months ago

You should then change the workflow to use "ClassMapper" as it is the proper repository. Although you would have to do that as this PR was closed.

RoseTheFlower commented 6 months ago

It works either way, and it's more consistent with your current README instructions.

shdwmtr commented 6 months ago

It is best to change it though In case this repository is modified and my initial repository no longer points here. Its a good idea to future proof it. And thanks for reminding me to update my README!