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

Update references and schema #8

Merged
merged 5 commits into from
Sep 17, 2024

Conversation

ds5678
Copy link
Contributor

@ds5678 ds5678 commented Sep 13, 2024

No description provided.

* MacOS tests require it
* Allows compiler to emit more optimized code
@nietras
Copy link
Owner

nietras commented Sep 15, 2024

Overall lgtm and thanks! Would you mind bumping version to 0.3.0 (I think)?

@nietras
Copy link
Owner

nietras commented Sep 15, 2024

There seems to be some weird build errors (lack of API) on linux, macos but not windows 🤔

Test project only.

@ds5678
Copy link
Contributor Author

ds5678 commented Sep 15, 2024

I think that might have fixed it.

@nietras
Copy link
Owner

nietras commented Sep 15, 2024

please don't remove global.json, fix version in that to latest dotnet 8 sdk

switch latestMajor to latestPatch

@ds5678 ds5678 force-pushed the update-references-and-schema branch from 4e4e034 to 3aabad1 Compare September 15, 2024 15:32
@ds5678
Copy link
Contributor Author

ds5678 commented Sep 15, 2024

Good now?

@ds5678
Copy link
Contributor Author

ds5678 commented Sep 15, 2024

Anything else I need to do?

@nietras
Copy link
Owner

nietras commented Sep 15, 2024

@ds5678 no I think it should be good, I'll try to look it over tomorrow on my PC and merge if all ok, then publish new version on nuget. :)

@nietras nietras merged commit 04ae73f into nietras:main Sep 17, 2024
6 checks passed
@nietras
Copy link
Owner

nietras commented Sep 17, 2024

@ds5678 merged, thank you. Next is update on nuget which may not be until tomorrow though.

@nietras
Copy link
Owner

nietras commented Sep 17, 2024

should be up on nuget now

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