Fork + github + npm
-
So, as most people are probably aware, I've been updating and submitting PRs for our youtube plugin. However, the original author is pretty much AWOL at this point, so I think it's probably time to fork it. Someone over at c.nodebb suggested this already. I'm sure they'd like to get access to the updates via npm.
Should I make a new github repo? Just ignore the original repo and go back to master? What?
Obviously it will need a new name. I was thinking nodebb-plugin-youtube-embed. But I guess I'm open to ideas.
-
@boomzilla said in Fork + github + npm:
Should I make a new github repo?
nah, keep the fork. makes it easier to merge back into the original if the author returns from being AWOL.
also gives a history link of the project.
@boomzilla said in Fork + github + npm:
Obviously it will need a new name. I was thinking nodebb-plugin-youtube-embed.
-
@boomzilla said in Fork + github + npm:
But I guess I'm open to ideas.
NodeBB-plugin-YouTube-wtfEmbed
? Are names case-sensitive? ;)
-
@Tsaukpaetra said in Fork + github + npm:
Are names case-sensitive?
no, because NPM forces lowercase for rasins.
-
@Tsaukpaetra said in Fork + github + npm:
NodeBB-plugin-YouTube-wtfEmbed ? Are names case-sensitive?
I'd rather keep the name generic rather than be site specific.
-
@boomzilla said in Fork + github + npm:
site specific.
Not necessarily site specific, but spawning-pool accrediting.
-
@Tsaukpaetra said in Fork + github + npm:
@boomzilla said in Fork + github + npm:
But I guess I'm open to ideas.
NodeBB-plugin-YouTube-wtfEmbed
? Are names case-sensitive? ;)real_escape_YouTube-plugin-NodeBB
-
@Lorne-Kates That one's deprecated. You mean
really_real_escape_YouTube-plugin-NodeBB_final_usethisone_2
.
-
Alternate suggestion:
fork_youtube_give_me_plugin
-
@Lorne-Kates said in Fork + github + npm:
real_escape_YouTube-plugin-NodeBB
@error said in Fork + github + npm:
really_real_escape_YouTube-plugin-NodeBB_final_usethisone_2
@Lorne-Kates said in Fork + github + npm:
fork_youtube_give_me_plugin
NodeBB plugin names have to start with
nodebb-plugin-
and be all lowercase ASCII and hyphens. (And maybe ASCII digits are ok? Dunno.)
-
@ben_lubar Needs more emoji.
-
@TDWTF-NodeBB-Development does anyone know if Discourse made any significant database changes?
-
@boomzilla said in Fork + github + npm:
Obviously it will need a new name. I was thinking nodebb-plugin-youtube-embed. But I guess I'm open to ideas.
nodebb-plugin-youtube-embedr. Now it's 'cooler'.
-
@Tsaukpaetra said in Fork + github + npm:
NodeBB-plugin-YouTube-wtfEmbed ?
nodebb-plugin-wtf-youtube-y-u-no-embed
?
-
@ben_lubar said in Fork + github + npm:
@TDWTF-NodeBB-Development does anyone know if Discourse made any significant database changes?
/Shrug. Beats me. Try it and find out?
-
@ben_lubar said in Fork + github + npm:
@TDWTF-NodeBB-Development does anyone know if Discourse made any significant database changes?
I love that we're helping other people to leave that shitheap of a forum.
-
@DoctorJones unfortunately they're coming to this shit heap of a forum?
-
@Arantor said in Fork + github + npm:
@DoctorJones unfortunately they're coming to this shit heap of a forum?
true, but at least the developers of this one aren't an arsehole about it.
-
@accalia true enough, but I think being an arsehole lead dev isn't necessarily the worst thing that could ever happen. Jeff is a particularly egregious case of reality distortion field more than he is arsehole.
-
@Arantor said in Fork + github + npm:
@accalia true enough, but I think being an arsehole lead dev isn't necessarily the worst thing that could ever happen. Jeff is a particularly egregious case of reality distortion field more than he is arsehole.
i thought the former led to the latter?
-
@accalia True, but you could have the latter without the former and it wouldn't be so bad
-
@Arantor said in Fork + github + npm:
Jeff is a particularly egregious case of reality distortion field more than he is arsehole.
There's a strong correlation between the two things.