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

Compiler warning 3370 fixed #914

Closed
wants to merge 2 commits into from

Conversation

Fxplorer
Copy link
Contributor

@Fxplorer Fxplorer commented Mar 29, 2024

replacement for #913

When building, was receiving compiler warning 3370
The use of ':=' from the F# library is deprecated. See https://aka.ms/fsharp-refcell-ops. For example, please change 'cell := expr' to 'cell.Value <- expr'.

Following the guidance, changes made to offending expressions.

All tests still showing green.

Before you go

Hi there! Thank you for your contribution to our project. To help maintain the quality of our codebase, our Continuous Integration (CI) system will automatically run several checks on your submission. To streamline the process, we kindly ask you to perform these checks locally before pushing your changes:

To execute all build scripts, please run:

dotnet fsi build.fsx

If you encounter any formatting issues, you can auto-correct them by running:

dotnet fantomas build.fsx src tests docs

Should any tests fail, please review and adjust your changes accordingly.

We appreciate your efforts to contribute and look forward to reviewing your pull request!

When building, was receiving `The use of ':=' from the F# library is deprecated. See https://aka.ms/fsharp-refcell-ops. For example, please change 'cell := expr' to 'cell.Value <- expr'.`

Following the guidance, changes made to offending expressions.
@Fxplorer Fxplorer marked this pull request as ready for review March 29, 2024 16:40
@Fxplorer Fxplorer closed this Mar 29, 2024
@Fxplorer Fxplorer deleted the compiler-warning-3370 branch March 29, 2024 16:44
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.

1 participant