Skip to content

DO NOT MERGE: Modify CHANGELOG.rst - #360

Closed
cclauss wants to merge 1 commit into
developfrom
DO-NOT-MERGE-Modify-CHANGELOG.rst
Closed

DO NOT MERGE: Modify CHANGELOG.rst#360
cclauss wants to merge 1 commit into
developfrom
DO-NOT-MERGE-Modify-CHANGELOG.rst

Conversation

@cclauss

@cclauss cclauss commented Apr 6, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
Member

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

@aaronliu0130

aaronliu0130 commented Apr 6, 2025

Copy link
Copy Markdown
Member

@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

cclauss commented Apr 6, 2025

Copy link
Copy Markdown
Member Author

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.

2 participants