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.

Sign up for GitHub

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

Jump to bottom

Made "/" separator compatible across all operative systems #410

Draft

gabridele wants to merge 6 commits into PennLINC:main from gabridele:separator_fix

Collaborator

gabridele commented Jan 17, 2025

Closes #368

Changes proposed in this pull request

all references to paths using '/' where changed to more universal solutions like os.sep or using Path.pathlib

gabridele added 6 commits

January 17, 2025 21:30


          made / separator compatible across OS's in cubids.py

5870dc2


          made / separator compatible across OS's in validator.py

9c31484


          made / separator compatible across OS's in workflows.py

2a5792a


          Merge branch 'main' into separator_fix

3fef890


          fixed linting issue

81d99e7


          fixed linting issue

46736c4

gabridele marked this pull request as draft

January 17, 2025 20:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet