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

Project maintenance updates #2763

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Project maintenance updates #2763

wants to merge 7 commits into from

Conversation

xperiandri
Copy link
Collaborator

Description

  • Migrate to default .gitignore
  • Updated Paket to 7.2.1
  • Update Fantomas to 6.2.1
  • Update FsDocs to 19.1.0
  • Update .NET SDK on CI to 6.0.414
  • Make .editorconfig more safe
  • boy scout rule: "leave the code behind in a better state than you found it" (fix warnings, obsolete members or code-style in the places you worked in)
  • Fake API guideline is honored

@Numpsy
Copy link
Contributor

Numpsy commented Oct 1, 2023

/home/runner/work/FAKE/FAKE> "dotnet"  fantomas src/app/ src/template/ src/test/ --recurse --check (In: false, Out: false, Err: false)
Unhandled exception. System.IO.DirectoryNotFoundException: Could not find a part of the path '/home/runner/work/FAKE/FAKE/--recurse'.

I think Fantomas 6 made the folder searching recursive by default, and deprecated the command line flag for it - so ---recurse can be removed from the invocation?

@xperiandri xperiandri force-pushed the project-maintenance branch 2 times, most recently from f40c601 to 047a1a1 Compare October 29, 2023 18:44
@Numpsy
Copy link
Contributor

Numpsy commented Nov 24, 2023

Is it worth updating Paket to v8 now, or leave that till later?

@xperiandri
Copy link
Collaborator Author

I would release this changes to v6 and then updated to v8

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.

None yet

2 participants