Skip to content

Conversation

@j-sawn
Copy link
Contributor

@j-sawn j-sawn commented May 20, 2025

Pull Request

This should fix the current issue with the templates not working (<-- needs to be fact-checked)

Changes Included

This should list all changes made in this merge request:

  • Addresses current issue with the template files, where they are not being used at all because it is not automated (which defeats the whole purpose of it)

Checklist

  • I have tested my code in my own container (using the Modelworks Dockerhub image) and it works (unapplicable)
  • Someone else other than me has tested my code in their own container (using the Modelworks Dockerhub image) and it works (unapplicable)
  • I have written documentation for my work somewhere in its files (should be straightforward enough but the pull request's summary should give you the information you need
  • Someone else has reviewed this pull request and deems it worthy to be merged
    • Reviewer Name: Matthew

Associated Issue

Other Notes

  • Again, should be easily reversible and should not meddle with pre-existing code
  • Apologies for the hassle!

@j-sawn j-sawn changed the title (DO NOT MERGE) Quickfix to Template Integration (GOOD TO MERGE) Quickfix to Template Integration Aug 8, 2025
@j-sawn
Copy link
Contributor Author

j-sawn commented Aug 8, 2025

Okay updates I tested this out in a personal repository (here) and it seems to work? So this should be good to merge now. Sorry for the wait!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants