search

pcdshub / btms-ui

Laser Hall Beam Transport Motion System User Interface
Other
0 stars 4 forks source link

Setup standard ECS GH workflows #9

Closed slactjohnson closed 2 months ago

slactjohnson commented 2 months ago

Description

Adds in standard ECS GH workflows.

Motivation and Context

Closes #8

How Has This Been Tested?

Will hopefully be tested when the workflows run on this PR....

tangkong commented 2 months ago

We transitioned to pyproject.toml for specifying test dependencies at some point. Try adding something like this to pyproject.toml

https://github.com/pcdshub/atef/blob/11b15c01dfafd9fe3c6d5ea39afd5844c071c339/pyproject.toml#L39-L40

ZLLentz commented 2 months ago

Wow, that's a lot of commits and a well earned "all checks have passed"

slactjohnson commented 2 months ago

Thanks! I'd appreciate a once over if you can to see if there's anything weird or missing in the CI. It passes, but that doesn't mean it's correct...

slactjohnson commented 2 months ago

@tangkong I've committed your suggestion to fix README.rst, but now it requires a review again. Can you approve when you get a chance so I can merge?

slactjohnson commented 2 months ago

Thank you!