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

Rename ASETProps-CE-1.4.ckan to ASETProps-CE-1.4.deleted #1245

Merged
merged 2 commits into from
Feb 12, 2017

Conversation

Dazpoet
Copy link
Member

@Dazpoet Dazpoet commented Feb 11, 2017

Follow-up on KSP-CKAN/NetKAN#5257

If we had KSP-CKAN/CKAN#1888 or KSP-CKAN/CKAN#1929 ready this would be less painful for our users who might now be left in a state where they cannot uninstall without the help of the CLI.

I'm honestly not sure how this will interact with the installation of the new ASET packages from KSP-CKAN/NetKAN#5254 but I'm ready to bet we will get an issuereport if it screws up horribly.

I invented a new file-extension for this because I felt like it and am tired. Hoping this means we won't pick it up as an installable mod, @techman83 might now?

@techman83
Copy link
Member

@Dazpoet netkan won't, but the testing won't pickup future clashes. Probably worth keeping 'frozen' as the convention unless there is a compelling reason otherwise.

@Dazpoet
Copy link
Member Author

Dazpoet commented Feb 12, 2017

I fixored it :)

@techman83
Copy link
Member

Oh gosh, I read that all wrong. My heads in NetKAN space today. So my former comment doesn't apply. However '.frozen' is a nomenclature we've used before, so it's still applicable.

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

Successfully merging this pull request may close these issues.

2 participants