Seq extensions: add trySkip

[knocte]

This way it is an alternative to Seq.skip: same but
non-partial function (that doesn't throw exceptions),
following same approach as tryHeadTail[1].

[1] fdbeaf2

[knocte]

@gdziadkiewicz hey do you mind helping me upgrade paket (and paket deps) from v6 to v8? I'm not a paket expert, I just replaced net6.0 with net8.0 and upgraded paket version (see last commit), but seems that is not enough (see build failure).

[gdziadkiewicz]

Hi @knocte , no problem, I will take a look tomorrow morning :)

[gdziadkiewicz]

Hi @knocte , let's keep the update and the trySkip separate. I prepared the update in #224 .

[knocte]

Lovely thanks; sounds good, I'll rebase as soon as 224 lands.

[gdziadkiewicz]

@knocte I'm working on making the Fable tests work after my changes, sorry for the wait

[gdziadkiewicz]

@knocte merged the other PR

[knocte]

@gdziadkiewicz hey, thanks! Rebased.

Now, it seems CI is failing because, even though your PR upgraded general things to .NET8.x(&F#v8.x), FSharp.Core seems to be still locked to 4.x.

How can I upgrade FSharp.Core to v8.x? I assume that I gotta change manually the version in packet.dependencies file, but I guess I gotta run something else after that change, to make it propagate to packet.lock file?

Thanks.

(BTW the PR can already be reviewed in case you have feedback, and I will address it too next time I update the PR, cheers.)

[knocte]

How can I upgrade FSharp.Core to v8.x? I assume that I gotta change manually the version in packet.dependencies file, but I guess I gotta run something else after that change, to make it propagate to packet.lock file?

Mmm, changing manually both files seemed to work. If this is ok lmk and I'll squash latest commit to get this ready to be merged.

[gdziadkiewicz]

I'm wondering if this is what users will expect. I would rather keep Seq.skip's behaviour for cases where it does not throw

@knocte WDYT?

[gdziadkiewicz]

Wouldn't implementation following Seq.skip's impl (https://github.com/dotnet/fsharp/blob/main/src/FSharp.Core/seq.fs#L1611) perform better? It would also allow us to stay on FSharp.Core 4.3.4 (as there would be no need for checked tail recursion) and not risk cutting off someone stuck on old FSharp.Core versions.