Skip to content

Import replace.fsx script (with git history) from fsx repo#237

Merged
knocte merged 18 commits intomasterfrom
wip/importReplaceScript
Apr 3, 2026
Merged

Import replace.fsx script (with git history) from fsx repo#237
knocte merged 18 commits intomasterfrom
wip/importReplaceScript

Conversation

@knocte
Copy link
Copy Markdown
Member

@knocte knocte commented Apr 3, 2026

No description provided.

knocte and others added 14 commits April 20, 2019 15:33
File.WriteAllText writes in utf8 (no-bom) by default but we need to keep
the original bom to prevent unnecessary diff between the incoming and
outgoing file.

Currently, output is always written in utf8 so keeping any other BOMs
(except Utf-8 bom) would result in invalid files.
replace: keep original utf8 bom

See merge request nblockchain/fsx!11
This way it focuses on one file only and will be much much faster
(we will use this mode in bump.fsx).

From nblockchain/geewallet@837457d
It was introduced in 155d0c4
and reverted by mistake in 6a34c7c.
See https://docs.microsoft.com/en-us/dotnet/fsharp/tools/fsharp-interactive/#using-the-fsi-object-in-f-code

(And we rename the InfraLib API related to this
to make the difference more clear.)
In preparation for nuget package upload of it.
Publish Fsdk as nuget package.
So that they can be used from the nuget pkg.
@knocte knocte force-pushed the wip/importReplaceScript branch from ba41023 to d413baf Compare April 3, 2026 20:23
@knocte knocte force-pushed the wip/importReplaceScript branch from d413baf to 333d796 Compare April 3, 2026 20:31
knocte added a commit to nblockchain/fsx that referenced this pull request Apr 3, 2026
@knocte knocte force-pushed the wip/importReplaceScript branch from 3d08557 to 0c57673 Compare April 3, 2026 21:00
@knocte knocte merged commit abf9b3b into master Apr 3, 2026
9 of 10 checks passed
@knocte knocte deleted the wip/importReplaceScript branch April 3, 2026 21:17
knocte added a commit to nblockchain/fsx that referenced this pull request Apr 3, 2026
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