FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

DO NOT MERGE: Modify CHANGELOG.rst by cclauss · Pull Request #360 · cpplint/cpplint · GitHub

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 commented Apr 6, 2025
edited
Loading

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 requested a review from aaronliu0130 April 6, 2025 19:33
cclauss closed this Apr 6, 2025
cclauss deleted the DO-NOT-MERGE-Modify-CHANGELOG.rst branch April 6, 2025 19:43
aaronliu0130 restored the DO-NOT-MERGE-Modify-CHANGELOG.rst branch April 6, 2025 20:03

Copy link
Copy Markdown
Member

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

aaronliu0130 commented Apr 6, 2025
edited
Loading

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 commented Apr 6, 2025

Copy link
Copy Markdown
Member Author

OK. I removed the three required tests.

cclauss closed this Apr 6, 2025
cclauss deleted the DO-NOT-MERGE-Modify-CHANGELOG.rst branch April 6, 2025 20:14
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL