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

Fixing bugs for HATS. #19

Merged
merged 18 commits into from
Jun 4, 2020
Merged

Fixing bugs for HATS. #19

merged 18 commits into from
Jun 4, 2020

Conversation

jpivarski
Copy link
Member

No description provided.

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

You can set fail-fast to false to allow all the jobs to run, rather that quitting as soon as one fails.

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

Looks like conda may be installed on Linux only.

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

Conda is pre-installed in Windows too, just not macOS: https://github.com/actions/virtual-environments/blob/master/images/win/Windows2019-Readme.md

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

You'll probably need shell: bash -l {0}, and it will really slow down your tests to preinstall conda each time. How about just testing conda on linux and windows, and then having a pip-based test elsewhere?

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

Keep in mind in multiple OS testing, Windows defaults to powershell, you need shell: bash to get the bash shell on Windows. (All OS's have powershell-core, so you can go all powershell, too)

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

conda init bash is the wrong thing to use for the setup-miniconda action - that needs the shell: bash -l {0} instead. Also conda init bash requires that you restart your shell, IIRC.

@jpivarski
Copy link
Member Author

Thanks for the help, @henryiii—I haven't been ignoring you; I just noticed all of these messages where you warned me of just about everything I stumbled over.

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

Conda is pre-installed on all OS's, including macOS. I'm discussing with GitHub Actions team, will have an update here soon.

@jpivarski
Copy link
Member Author

You can set fail-fast to false to allow all the jobs to run, rather that quitting as soon as one fails.

The fail-fast is new to me in GitHub Actions (if it was possible in Travis or Azure, I didn't know about it), but I like it. Usually, when a commit is wrong, it's wrong in all platforms because it's a coding bug. (GHA's flake8 catches more errors than mine, and I need to figure out how to get that version! Oh, 3.7 vs 3.8... maybe that's it.)

Occasionally, I really do need it to not fail-fast, such as a recent search on scikit-hep/uproot3#493 for the struct syntax for that platform-dependent long type, and I'll turn off fail-fast if I'm in one of those situations.

@henryiii
Copy link
Member

henryiii commented Jun 4, 2020

They've changed the defaults and names, but otherwise, GHA is pretty much identical to Azure. And it was also possible to enable fail-fast in Travis, but don't remember the name; CI systems like fail-fast, after all. :)

GitHub CI will fail on a multiline command if a command in the middle fails, which is fantastic, whereas Azure will fail only if the final command fails, which is awful - they said it was too late to change the default in Azure so they only changed it when they developed GHA.

@jpivarski
Copy link
Member Author

The Uproot HATS is done, so I'll close off this PR and give it a version number.

@jpivarski jpivarski marked this pull request as ready for review June 4, 2020 18:24
@jpivarski jpivarski merged commit d6956b6 into master Jun 4, 2020
@jpivarski jpivarski deleted the jpivarski/fixing-bugs-for-hats branch June 4, 2020 18:25
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