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

Remove outdated .Build.cs template #346

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

Th3Fanbus
Copy link
Contributor

Docs already reference the Alpakit template in the master branch of the SML repo, so there's no need to maintain a copy of the template that is bound to become outdated over time. Also, update the name to replace to PLUGIN_NAME as used in the Alpakit template.

Docs already reference the Alpakit template in the master branch of
the SML repo, so there's no need to maintain a copy of the template
that is bound to become outdated over time. Also, update the name
to replace to `PLUGIN_NAME` as used in the Alpakit template.
Comment on lines 74 to 81

Within this new file you will need to add the following configuration text.

Make sure to replace all instances of `YourModReference` with your actual mod reference.
Make sure to replace all instances of `PLUGIN_NAME` with your actual mod reference.

The latest version of this template can be found in the
https://github.com/satisfactorymodding/SatisfactoryModLoader/blob/master/Mods/Alpakit/Templates/CPPAndBlueprintBlank/Source/PLUGIN_NAME/PLUGIN_NAME.Build.cs[Starter Project Repository as an Alpakit template].

Copy link
Member

Choose a reason for hiding this comment

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

Could you reorder+rephrase this so it's saying to visit the template file, then telling you how to edit it? Right now it's in reverse. I'm on mobile so cant use the Suggest feature easily

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I swapped the paragraphs' order, I think it's reasonable enough but I'd like to know what your thoughts are.

Hopefully, this is what Robb meant.
@budak7273 budak7273 merged commit 272273e into satisfactorymodding:Dev Oct 24, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🚢 Released
Development

Successfully merging this pull request may close these issues.

2 participants