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 Cake.Args.yml #1498

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cake-contrib-bot
Copy link
Contributor

This pull request was created by a tool: Cake.AddinDiscoverer version 3.63.0

Resolves #1497

@augustoproiete
Copy link
Member

As per @pascalberger's suggestion, I reached out to the creator of the Cake.ArgumentHelpers to see what his thoughts are in reviving his addin and possibly merging Cake.Args into it, instead of having two separate addins with very similar goals.

I'll wait a few days to see if I get a response. Marking as blocked for now.

@augustoproiete augustoproiete added the merge blocked PR should not be merged yet label Jan 23, 2021
@augustoproiete
Copy link
Member

Got an initial response from Adam and he'll let me know if he can find some time to do the following:

As soon as possible

  • Update Cake reference libraries v0.33.0
  • Multi-target net461 and netstandard2.0
  • Update the categories of the aliases (PR from @pascalberger)
  • Merge the method ArgumentOrDefault from Cake.Args into Cake.ArgumentHelpers
  • Release Cake.ArgumentHelpers v0.4.0 to nuget.org

After we release Cake v1.0.0

  • Update Cake reference libraries to v1.0.0
  • Include net5.0 target (and keep net461, netstandard2.0)
  • Release Cake.ArgumentHelpers v0.5.0 to nuget.org

ps: Did I forget anything?

@augustoproiete augustoproiete force-pushed the add_Cake.Args.yml_2021_01_23_00_06_15 branch from dd2ca38 to ef7ccd1 Compare February 19, 2021 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge blocked PR should not be merged yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Cake.Args.yml
2 participants