-
-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Come up with a way to templatize and automatically generate the Jobs config #41
Comments
We're going to have to do this for 1.16, since we're at the point where we have to manually develop our Jobs settings (since we've diverged so radically from upstream). |
One way to do this, is to have a file for every release of Minecraft, with just the blocks it added to the game listed. Then, when we update to a new version, all we'll need is a list of the new blocks in a new file. Easy. |
We could have a base value for each block. We could have a modifier for each job, which increases or decreases the base values for that job. We could also have job-specific overrides. |
We can't limit each block to only one job -- Miner gets paid by breaking the same blocks that Builder gets paid for placing, for example. |
Possible syntax:
|
I can parse the files and store them in an object, and then use that object to generate the configs Jobs expects to have. |
We might be able to mostly derive the values for Jobs from those of mcMMO. This should give us coverage for edge-cases, since the mcMMO dev is proactive about catching and correcting those. (The Jobs dev, at least the last time I interacted with him, is actively hostile to patching them.) It we go this route, most jobs will be combinations of various mcMMO skills. Other jobs, like Crafter, will still have to be custom. |
Another option could be to make our own jobs plugin. We would have to keep it simple, but there are some significant benefits to be had with this such a thing. Benefits:
Caveats:
Notes:
|
The Jobs config is ridiculously large. We cannot hope to properly manage it as it is. If we could create a template file, then we could create a script that would perfectly generate most of the Jobs config for us.
The text was updated successfully, but these errors were encountered: