Pull request in my repo! Advise me

[Jenkler]

I got a pull request with no description. Is that patch needed and why?

Update plugin.js by TheWorms · Pull Request #1 · Jenkler/nodebb-plugin-postlink

GitHub (github.com)

@julian

[baris]

It should be const meta = require.main.require('./src/meta');

[Jenkler]

Ok. but why does it work with only ./meta ? What is the issue with using ./meta? Seams to find it anyway

[PitaJ]

The change the guy suggested will actually break your plugin I think. The change he made should have been ../meta instead of ./src/meta. This is because NodeBB internal changes were made resulting in moved modules. Right now it has a backwards compatibility later but eventually plugins should switch to using require.main.require which will work the same regardless.

With require.main.require you need to change it from ./database to ./src/database.

[Jenkler]

Ohh, crap. I missed the "require.main.require" part Then its better to to patch this

[PitaJ]

@Jenkler the actual PR does not have require.main.require so it will break your plugin.

[Jenkler]

@baris said in Pull request in my repo! Advise me :

const meta = require.main.require('./src/meta');

Fixed correct require for meta · Jenkler/nodebb-plugin-postlink@c2f87f7

A plugin that adds a prefix on postlinks. Contribute to Jenkler/nodebb-plugin-postlink development by creating an account on GitHub.

GitHub (github.com)

[Jenkler]

lol, does it look OK now?

[Jenkler]

@PitaJ said in Pull request in my repo! Advise me :

@Jenkler the actual PR does not have require.main.require so it will break your plugin.

Yeah, found out that when it was to late But now it should be okay Thanks!

[Jenkler]

I need to watch out for my trigger fingers