Cat devel merger #85
Cat devel merger #85
Conversation
There was a problem hiding this comment.
Moved to top level of directory so there's not a separate docs-only environment for developers. Can add more environment variables for development here as needed.
There was a problem hiding this comment.
Combination of the old build.yml (which built with hatch) and python-package.yml (which tested with pytest).
There was a problem hiding this comment.
I had build.yml separate because it is used when making releases ( "on: release:"). This commit changes that function entirely, so I can't accept this change.
There was a problem hiding this comment.
The recommended way of doing this is to invoke build action from whatever other yml you want to do a build. We should be moving towards smallish action ymls that can be invoked in different places.
There was a problem hiding this comment.
also the workflow_dispatch is required so we can run this action manually from github when needed
There was a problem hiding this comment.
Reverted build.yml to its original version and renamed the other test build-and-pytest.yml
There was a problem hiding this comment.
Test to check if the pyinstaller run fails for the GUI. GUI code not added yet, but I'm adding this with the tests so it runs when I do add the GUI code.
There was a problem hiding this comment.
Combined with build.yml (see above)
There was a problem hiding this comment.
Fixed a typo and added instructions for specifying which Python hatch should use (since the system python at GBO is way out of date)
There was a problem hiding this comment.
Placeholder file that needs to be filled out with more contribution information
There was a problem hiding this comment.
Info about the rich text printing I added
Hopefully I don't have to close this again! This is all the documentation and test changes. I'll go through and leave comments on changes that might be confusing.