mne-cpp icon indicating copy to clipboard operation
mne-cpp copied to clipboard

WIP [ENH] Ci linux using cl scripts

Open juangpc opened this issue 4 years ago • 2 comments

The objective of this PR is to, soon, be able to run, with exactly the same script, jobs locally and in the GitHub actions. There is still a bit of work to do.

  • [x] Understand completely what do all the different GitHub workflows do.
  • [ ] Create a single multiplatform script for each workflow.
  • [x] I'd like to discuss if we should just use python for all these and forgive us all of using bash scripting.
  • [ ] Standardize how to treat input arguments for all scripts. Source all the functions into one single script.
  • [ ] validate results.
  • [ ] ready to merge.
  • [ ] Harmonize the use of windows versions (I see sometimes 2016 and some other times 2019).

juangpc avatar Aug 03 '21 14:08 juangpc

@juangpc Is the goal of the PR to introduce a build script so that we can more easily build the project locally exactly just as we do in the CI? This would probably be a good idea since it will make the CI workflow yaml files easier to read.

I am just a bit confused because you mentioned "jobs" in the PR description. We should not try to replace every job in all the workflows by our own scripts. Workflows can house jobs, jobs are composed out of multiple steps, which again can be represented by Github actions. There are a lot of great Github actions available and we should not try to replace them with scripts written by us. Just take the Install Qt action as an example, rewriting this on our own is not a good idea. Not sure if this was your intention, but if it was we should discuss.

LorenzE avatar Oct 09 '21 10:10 LorenzE

I think I understand what you mean, and in principle I agree with you. But, let's take your example: i already have a script that builds from qt from sources . I had to do it because I needed to understand why codecov script was not working. This was a few weeks ago, you probably remember. That time the task of fixing codecov figures, was,in my opinion unecesarily cumbersome due not having the script ready.

It is a bit of a philosophy approach. Do you want a ci that replicates your duties automatically in the cloud or do you want it to independent process that checks your code.

I'm more inclined to make a ci something that you work with, everyday, locally. Compilation steps, testing, code coverage, I'd like to know locally and then replicate automatically in the cloud before merging.

So the next question is. How do you do that with JS GitHub actions?

So yes. Let's discuss.

juangpc avatar Oct 09 '21 13:10 juangpc