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

WIP - Converted GitVersionHelper to FAKE 5 module #1988

Merged
merged 5 commits into from
Jun 9, 2018

Conversation

landy
Copy link
Contributor

@landy landy commented Jun 8, 2018

No description provided.

@landy landy changed the title Converted GitVersionHelper to FAKE 5 module WIP - Converted GitVersionHelper to FAKE 5 module Jun 8, 2018
Copy link
Member

@matthid matthid left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Looks good overall, just some minor things to cleanup :)

open Fake.IO.Globbing
open Fake.Core

[<CLIMutable>]
Copy link
Member

Choose a reason for hiding this comment

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

Please remove the CLIMutable.

ToolPath : string
}

let GitversionDefaults = {
Copy link
Member

Choose a reason for hiding this comment

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

We can probably make this private (or internal)

}

type GitVersionProperties = {
Major : int;
Copy link
Member

Choose a reason for hiding this comment

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

Can we change the formatting to be less intended and remove the ;?

@landy
Copy link
Contributor Author

landy commented Jun 9, 2018

fixed suggestion

@matthid matthid changed the base branch from master to release/next June 9, 2018 13:11
@matthid
Copy link
Member

matthid commented Jun 9, 2018

Can you merge release/next or do a dummy commit to restart the CI on a green branch (sorry).

@matthid
Copy link
Member

matthid commented Jun 9, 2018

Thanks!

@matthid matthid merged commit 52411c7 into fsprojects:release/next Jun 9, 2018
@matthid
Copy link
Member

matthid commented Jun 9, 2018

Congrats to your first contribution :)

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