Skip to content

Conversation

@kMutagene
Copy link
Member

No description provided.

@kMutagene kMutagene requested a review from omaus April 22, 2025 15:11
@kMutagene
Copy link
Member Author

kMutagene commented Apr 22, 2025

@omaus can you review and release if it is fine?

I need this in a downstream repo (BioFSharp)

Copy link
Member

@omaus omaus left a comment

Choose a reason for hiding this comment

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

Just a quick question about versioning, Seq.chunkBy seems absolutely plausible for me. 👍🏻

<PackageReference Include="Fake.Api.Github" Version="6.0.0" />
<PackageReference Include="Fake.Core.Process" Version="6.0.0" />
<PackageReference Include="Fake.Core.ReleaseNotes" Version="6.0.0" />
<PackageReference Include="Fake.Core.Target" Version="6.0.0" />
Copy link
Member

Choose a reason for hiding this comment

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

Just a quick question: Wasn't it the case that we wanted to keep all version tags exact? (I.e., Version="[6.0.0]")

Copy link
Member Author

Choose a reason for hiding this comment

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

I think in the case of the build project it does not matter, but feel free to pin

@omaus
Copy link
Member

omaus commented Apr 22, 2025

@kMutagene

@omaus omaus merged commit dcb08c4 into main Apr 23, 2025
2 checks passed
@omaus omaus deleted the kMutagene-updates branch April 23, 2025 17:51
@omaus omaus restored the kMutagene-updates branch April 23, 2025 17:51
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.

3 participants