Feature: User Configured Jobs directory

[AlphaRomeoMike]

This PR link to #11 to add user configurable path for adonis_resque

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	@​adonisjs/​prettier-config@​1.3.0 ⏵ 1.4.5	+2		+12	+5
	@​adonisjs/​eslint-config@​1.3.0 ⏵ 1.3.1
	@​types/​node@​22.5.0 ⏵ 22.16.0			+1
	node-resque@​9.3.5 ⏵ 9.3.8	-1			-3
	@​adonisjs/​core@​6.12.1 ⏵ 6.19.0	+1		+1	+1
	@​adonisjs/​tsconfig@​1.3.0 ⏵ 1.4.1	+3		+13	+7
	tsup@​8.2.4 ⏵ 8.5.0	+1	+5
	typescript@​5.5.4 ⏵ 5.8.3			+1
	croner@​8.1.1 ⏵ 8.1.2
	@​adonisjs/​assembler@​7.7.0 ⏵ 7.8.2	+3			+5	+1
	nock@​13.5.5 ⏵ 13.5.6
	@​poppinss/​utils@​6.7.3 ⏵ 6.10.0	+1			+6
	eslint@​8.57.0 ⏵ 8.57.1

View full report

[AlphaRomeoMike]

@shiny will it be an issue if I add a new package to the dependencies of this lib? I have added fast-glob to allow glob processing. I have also bumped a few versions of packages.

[shiny]

Thank you for your contribution! Regarding this PR, I don't think the new dependency is necessary — fsImportAll already supports a filter method. (Just a side note: @poppinss/utils is a core AdonisJS library.)
In general, we try to avoid introducing extra dependencies unless truly needed.

Secondly, the name jobsGlob feels a bit too geeky. Do you have a more elegant or intuitive alternative in mind?

[AlphaRomeoMike]

Thank you for your contribution! Regarding this PR, I don't think the new dependency is necessary — fsImportAll already supports a filter method. (Just a side note: @poppinss/utils is a core AdonisJS library.) In general, we try to avoid introducing extra dependencies unless truly needed.

Secondly, the name jobsGlob feels a bit too geeky. Do you have a more elegant or intuitive alternative in mind?

Thanks for the feedback here @shiny.

• I will take a look at the filter option
• If the name is too geeky, I think jobsPath will be a better alternative.

Let me know if you find that suitable

[AlphaRomeoMike]

@shiny kindly review now, I am now using the path instead of the using glob matching

[AlphaRomeoMike]

@shiny any update?

[shiny]

We can skip glob support for now and only allow a custom directory to be specified.
Also, we shouldn't use filter to enforce that files end with jobs. this could even break the examples.

[AlphaRomeoMike]

@shiny I have updated the path, can you kindly check now?