Skip to content

Comments

Build modernisation - replace paket and fake with simple fsi build#34

Merged
davidglassborow merged 10 commits intofsprojects:mainfrom
davidglassborow:build-modernisation
May 10, 2025
Merged

Build modernisation - replace paket and fake with simple fsi build#34
davidglassborow merged 10 commits intofsprojects:mainfrom
davidglassborow:build-modernisation

Conversation

@davidglassborow
Copy link
Collaborator

@davidglassborow davidglassborow commented Apr 25, 2025

#33

@dsyme
Copy link
Contributor

dsyme commented Apr 26, 2025

Great work!

@dsyme
Copy link
Contributor

dsyme commented Apr 26, 2025

@davidglassborow would you like to become co-maintainer on this repo with me?

@davidglassborow
Copy link
Collaborator Author

davidglassborow commented Apr 26, 2025

@dsyme Yes happy to be co-maintainer. You happy with my use of fsi for the build process ?

@dsyme
Copy link
Contributor

dsyme commented Apr 26, 2025

You happy with my use of fsi for the build process ?

Seems fine to me!

@davidglassborow davidglassborow marked this pull request as ready for review April 26, 2025 14:53
@davidglassborow
Copy link
Collaborator Author

@dsyme I think this is ready for review/merge, not sure what the situation is with the expired nuget token

@davidglassborow davidglassborow requested a review from dsyme April 26, 2025 14:55
@davidglassborow
Copy link
Collaborator Author

@dsyme nudge

@davidglassborow davidglassborow merged commit 102d3e4 into fsprojects:main May 10, 2025
2 checks passed
@davidglassborow davidglassborow changed the title WIP: Build modernisation Build modernisation - replace paket and fake with simple fsi build May 10, 2025
@davidglassborow davidglassborow deleted the build-modernisation branch May 10, 2025 14:03
@dsyme
Copy link
Contributor

dsyme commented Jun 24, 2025

@davidglassborow My apologies, running through old email notifications I just noticed this

Do you need the nuget token updated?

You can email me direct - my handle at github.com

@davidglassborow
Copy link
Collaborator Author

Nope all good, the comment in the action was misleading, the token is fine

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