Skip to content
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

Add nos17:creator-pack-02 #52

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

noah-severyn
Copy link
Contributor

No description provided.

@noah-severyn
Copy link
Contributor Author

The files installed correctly via command line. Tested in game.

@memo33
Copy link
Owner

memo33 commented Dec 7, 2024

Please take a look at the lint output.

@noah-severyn
Copy link
Contributor Author

🤦‍♂️

Comment on lines +33 to +57
- assetId: "dk1-bowling-lanes"
include:
- "\\.SC4Model$"
- assetId: "dk1-burger-chef"
include:
- "\\.SC4Model$"
- assetId: "dk1-convenience-stores-1"
include:
- "\\.SC4Model$"
- assetId: "dk1-convenience-stores-2"
include:
- "\\.SC4Model$"
- assetId: "dk1-donut-shops"
include:
- "\\.SC4Model$"
- assetId: "dk1-king-and-queen"
include:
- "\\.SC4Model$"
- assetId: "dk1-retro-kroger"
include:
- "\\.SC4Model$"
- assetId: "dk1-small-restaurants"
include:
- "\\.SC4Model$"

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Including files directly from foreign assets isn't what I'd usually do (see #40).

Would it be acceptable for you to add the dk1 packages as dependencies instead? This would imply that the dk1 lots gets installed, too, when your creator pack package is installed.

The alternative would be to split off the models from the dk1 packages into separate packages, so you only depend on the models without the lots.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Including files directly from foreign assets isn't what I'd usually do (see #40).

Would it be acceptable for you to add the dk1 packages as dependencies instead? This would imply that the dk1 lots gets installed, too, when your creator pack package is installed.

The alternative would be to split off the models from the dk1 packages into separate packages, so you only depend on the models without the lots.

Sure thing. I originally was not excited to make the models separate packages, but it's probably the best route to take now.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which of the two approaches do you prefer? Do you plan to implement it? Otherwise, I'll try to make the changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which of the two approaches do you prefer? Do you plan to implement it? Otherwise, I'll try to make the changes.

Sorry I wasn't clear. The second options. Not a huge fan of including them as dependencies as that would also install dk1's lots which I do not want to include. I will remake them as model/dependency packages, probably sometime within the next week or so.

@memo33 memo33 marked this pull request as draft December 29, 2024 19:33
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