Skip to content

Add a simple CI#39

Open
jklymak wants to merge 5 commits into
smerckel:masterfrom
jklymak:ci-make
Open

Add a simple CI#39
jklymak wants to merge 5 commits into
smerckel:masterfrom
jklymak:ci-make

Conversation

@jklymak

@jklymak jklymak commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

This just adds a simple CI that will run the tests and build the docs for pull requests.

OK it also makes some Mac and windows compatibility fixes.

Bumps python to 3.11 since date time.UTC requires that.

@jklymak

jklymak commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Note I'm getting the Mac-os errors locally as well on my Mac. Not sure about the windows ones. The 3.10 is the fact that python 3.10 module 'datetime' has no attribute 'UTC'

@jklymak

jklymak commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

PR now bumps python to 3.11.

@jklymak

jklymak commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

OK, this adds a fix for macOS. Not 100% sure it's correct, but macOS you can't just overwrite an existing file.

@jklymak

jklymak commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

OK and a couple of windows fixes.

@jklymak

jklymak commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

@smerckel if you do not want CI, consider the other two changes, but I think CI is pretty useful (unless you have a bunch of machines!)

@smerckel

smerckel commented Jul 4, 2026

Copy link
Copy Markdown
Owner

CI is fine. I will review the commits one of these days.

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