Skip to content

“Inactive” workflow: bump operations to 100 #25455

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
Merged
merged 1 commit into from
Mar 14, 2023
Merged

“Inactive” workflow: bump operations to 100 #25455

merged 1 commit into from
Mar 14, 2023

Conversation

rcomer
Copy link
Member

@rcomer rcomer commented Mar 14, 2023

PR Summary

As #25424. Increase available operations so more issues are labelled on the next run.

PR Checklist

Documentation and Tests

  • Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • New plotting related features are documented with examples.

Release Notes

  • New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

@rcomer
Copy link
Member Author

rcomer commented Mar 14, 2023

Has something changed with Azure? I’m sure it respected the [skip ci] in the commit message quite recently.

@oscargus
Copy link
Member

It at least has something that resembles something that may skip it...

matplotlib/azure-pipelines.yml

Line 30 in 43c35e7

condition: or(contains(variables.log, '[skip azp]'), contains(variables.log, '[azp skip]'), contains(variables.log, '[skip ci]'), contains(variables.log, '[ci skip]'), contains(variables.log, '[ci doc]'))

@rcomer
Copy link
Member Author

rcomer commented Mar 14, 2023 •
edited
Loading

If I click through to the Azure skip task, it appears to be checking the message from the last commit on main 🤔

Edit: maybe it’s just because my branch was not up-to-date with main?

@jklymak jklymak merged commit 08f75ac into matplotlib:main Mar 14, 2023
@QuLogic QuLogic added this to the v3.8.0 milestone Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oscargus oscargus oscargus approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v3.8.0
Development

Successfully merging this pull request may close these issues.
4 participants
@rcomer @oscargus @QuLogic @jklymak