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 Kopernicus Bleeding Edge repository #2159

Merged
merged 1 commit into from
Sep 20, 2020
Merged

Add Kopernicus Bleeding Edge repository #2159

merged 1 commit into from
Sep 20, 2020

Conversation

R-T-B
Copy link
Contributor

@R-T-B R-T-B commented Sep 19, 2020

People might want this... or not. Who knows?

People might want this...  or not.  Who knows?
@HebaruSan
Copy link
Member

That's the beauty of opt-in, if they don't want it, they don't have to have it. 👍

@HebaruSan
Copy link
Member

I tried downloading the .tar.gz, and I noticed a few possible syntax/validation issues with the .ckan files. Is there a place where I can submit fixes?

@R-T-B
Copy link
Contributor Author

R-T-B commented Sep 20, 2020

I tried downloading the .tar.gz, and I noticed a few possible syntax/validation issues with the .ckan files. Is there a place where I can submit fixes?

Sure, surprised it wasn't worse off as i basically best-guessed it until it worked. I'll upload it to a github repo, just a moment.

@R-T-B
Copy link
Contributor Author

R-T-B commented Sep 20, 2020

https://github.com/R-T-B/CKAN-meta-dev-Kopernicus_BE

Just make a PR here and I'll push it straight to my server.

@HebaruSan
Copy link
Member

Awesome, see R-T-B/CKAN-meta-dev-Kopernicus_BE#1

@HebaruSan
Copy link
Member

Alright, let's give this a try!!

@HebaruSan HebaruSan merged commit 2034f93 into KSP-CKAN:master Sep 20, 2020
@R-T-B
Copy link
Contributor Author

R-T-B commented Sep 20, 2020

Lookin good, thanks for the help with the syntax! I just got done learning ModuleManager syntax recently so it's easy to get things scrambled... lol

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