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

CI: Add style conditional #10723

Merged
merged 3 commits into from
Jun 7, 2022
Merged

CI: Add style conditional #10723

merged 3 commits into from
Jun 7, 2022

Conversation

larsoner
Copy link
Member

@larsoner larsoner commented Jun 7, 2022

One easy step from #10704: make Azure steps depend on the style step succeeding. This first commit adds a style error, so hopefully what we see is red CI with only Azure running the style tests, not everything. Then I'll push a commit to remove the style error, then hopefully we see all green Azure.

@larsoner
Copy link
Member Author

larsoner commented Jun 7, 2022

Looks like the dependencies worked:

Screen Shot 2022-06-07 at 5 55 09 PM

I'll merge once CIs come back happy

@mmagnuski
Copy link
Member

then hopefully we see all green Azure.

Eric, I think you meant "green pAztures". ;)

@mmagnuski
Copy link
Member

Jokes aside, this is really cool!

@larsoner larsoner merged commit 1df52a0 into mne-tools:main Jun 7, 2022
@larsoner larsoner deleted the cicond branch June 7, 2022 23:05
larsoner added a commit to larsoner/mne-python that referenced this pull request Jun 8, 2022
* upstream/main:
  MRG: In Report, allow to add multiple figure or image elements to the same content block ("section") (mne-tools#10694)
  CI: Add style conditional (mne-tools#10723)
  Fix coreg tutorial typos (mne-tools#10716)
  MAINT: Deal with nibabel deprecation (mne-tools#10719)
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