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 Heli #143

Merged
merged 4 commits into from
Nov 10, 2022
Merged

Remove Heli #143

merged 4 commits into from
Nov 10, 2022

Conversation

Lazerbeak12345
Copy link
Collaborator

We had to remove this mod because of legal issues.

See also #134

We had to remove this mod because of legal issues.

See also #134
Copy link
Collaborator

@dacmot dacmot left a comment

Choose a reason for hiding this comment

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

You also need to remove the submodule in builder/mods_src (just git rm does it), and run the update script to update the mod_sources.txt.

@Lazerbeak12345 Lazerbeak12345 requested a review from dacmot November 9, 2022 04:21
@dacmot
Copy link
Collaborator

dacmot commented Nov 9, 2022

That last commit undid all the changes of the first.

Before running the script, you need to remove the submodule.

@Lazerbeak12345
Copy link
Collaborator Author

ooof. yeah lol. i'll try again.

@Lazerbeak12345 Lazerbeak12345 merged commit 98e09e5 into main Nov 10, 2022
@Lazerbeak12345 Lazerbeak12345 deleted the remove-heli branch November 10, 2022 15:17
@Ygarr
Copy link

Ygarr commented Nov 10, 2022

Oh no((

@dacmot
Copy link
Collaborator

dacmot commented Nov 10, 2022

Yeah, this sucks... I wish we could keep it, but the main developer is MIA so we can count on a licence change anytime soon.

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.

3 participants