Skip to content

Conversation

@cclauss
Copy link
Member

@cclauss cclauss commented Apr 6, 2025

A test for
#359

  • The first commit should NOT run the REQUIRED tests.
  • The workflow_dispatch should run the REQUIRED tests.

Worked as expected. Closing.


  1. Go to https://github.com/cpplint/cpplint/actions/workflows/ci.yml
  2. At the left of the window, click Tests.
  3. At the right of the window, click the Run workflow drop-down menu.
  4. Select the desired branch.
  5. Click the Run workflow button.

https://github.com/cpplint/cpplint/actions/runs/14295936443

@cclauss cclauss requested a review from aaronliu0130 April 6, 2025 19:33
@cclauss cclauss closed this Apr 6, 2025
@cclauss cclauss deleted the DO-NOT-MERGE-Modify-CHANGELOG.rst branch April 6, 2025 19:43
@aaronliu0130 aaronliu0130 restored the DO-NOT-MERGE-Modify-CHANGELOG.rst branch April 6, 2025 20:03
@aaronliu0130
Copy link
Member

Reopening to check out which status checks display: the one on push or the one on dispatch

@aaronliu0130
Copy link
Member

aaronliu0130 commented Apr 6, 2025

@cclauss ...yeah, it's only the push ones that display; I just re-dispatched the test workflow. So no, the workflow dispatch things has no effect on status checks.

@cclauss
Copy link
Member Author

cclauss commented Apr 6, 2025

OK. I removed the three required tests.

@cclauss cclauss closed this Apr 6, 2025
@cclauss cclauss deleted the DO-NOT-MERGE-Modify-CHANGELOG.rst branch April 6, 2025 20:14
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.

3 participants