-
Notifications
You must be signed in to change notification settings - Fork 8
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
The future of this project #20
Comments
Hi, thank you, I appreciate my project was helpful. Indeed I have had no time to work on any of my hobby projects, hence this was left few versions behind the Pulumi schema. I'd welcome any PR to keep the project up-to-date. Maintenance should be fairly easy to keep the version in line with Pulumi as it may just need to bump up the version and kick another build, this could also be automated so no human intervention is required unless there are errors (rare). Happy to jump on a call to show you around the project. Thank you. |
Hi, thanks for the answer. As for automation of version upgrades, do you think we can do it with GH Actions? Or do you have some other CI/CD tool in mind? I would be happy to contribute to that as well. But I guess you might also need to spend some time on it. I can't fully implement it myself, obviously. |
Hi @mvsmal, there is already an Azure DevOps pipeline: https://github.com/UnoSD/Pulumi.FSharp.Extensions/blob/master/FAKE.yml But you will notice that the pipeline is pretty thin as it merely invokes the FAKE script, I wanted to make it as portable as possible and as much F# as possible everywhere; this means that, if you want to create a GH Actions pipeline, it should be a few minutes' work. Happy to merge the PR for GH Actions, however, if you want to add Docker, you just need to create a new pipeline that exposes the PROVIDER env variable "Docker" and that would just work, see here: https://github.com/UnoSD/Pulumi.FSharp.Extensions/blob/master/build.fsx let getProvider args =
match Map.tryFind "PROVIDER" args |>
Option.bind (function | Argument p -> Some p | _ -> None) |>
Option.orElse (Environment.environVarOrNone "PROVIDER") |>
Option.orElse (Environment.environVarOrNone "BUILD_DEFINITIONNAME") with
| Some "All" -> "*"
| Some p -> p
| _ -> failwith "Missing provider" |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Hey @UnoSD, I've been maintaining a fork over at cagyirey/Pulumi.FSharp.Extensions. I'd love to contribute to the core project. (especially to have PR reviewers) but it seemed like this repo was dead. Would love to chat more about the possibility of getting some of my work polished and merged. Some of the things I've been working on:
|
hi @cagyirey , thank you for working on this fork, I appreciate this project getting some contribution as I still believe it has great value. For personal reasons, I have not been able to work on it myself, but I'd welcome PRs and I'd be happy to integrate changes and push to NuGet to keep it alive. |
My first PR has been open since April :( Fully understand that you've got a lot of life things going on, but a lot of people are blocked right now from receiving package updates. With respect, it may be best to pass the torch to someone who currently has the capacity, or open the project up to maintainers. |
@cagyirey I will try and check out the PR as soon as I have a chance, apologies if I missed it. I am more than happy to open the project to maintainers, if you are happy to onboard. |
@mvsmal reviewing it now, apologies |
@UnoSD Would you accept a PR of @cagyirey's fork? Just playing around locally, I merged in latest main from this repo into theirs. Minor issues, pretty much just merging in the CustomResourceOptions CE. The big changes are that it looks like it publishes to a private NuGet feed, but we can sort all that out. What do you think, would you be open to taking that PR? |
hi @NatElkins, of course. If you would also like to help review, that would be awesome. @cagyirey can you please submit the PR? |
Hi there,
First of all I would like to thank you so much for this amazing project. We have been using the libraries quite heavily in our projects. I'd say that we strongly depend on them nowadays.
Unfortunately, looking at the commits history and the open issues it is obvious the you don't have much time for maintenance.
I was wondering what is the future of this project? Are you planning to put effort into it or you are going to abandon it completely?
If my team offers support on maintenance, how should we better do that? Would you have time to review our PRs?
Or maybe you can add me/us as collaborators so we can continue keeping the project up to date? In this case I would highly appreciate if you find time for an introduction call, just to go over the project structure and code style conventions. We don't want to spoil all the hard work you've done. We don't want to fork the project and create duplicates in the Nuget repository, we'd rather contribute to the original version.
Please let me know your thoughts. I'd be happy to discuss any details.
The text was updated successfully, but these errors were encountered: