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

Excluding self check run and same kind check runs when waiting for checks #638

Conversation

CatChen
Copy link
Owner

@CatChen CatChen commented Aug 28, 2022

No description provided.

Copy link
Owner Author

CatChen commented Aug 28, 2022

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ESLint doesn't pass. Please fix all ESLint issues.

@@ -1,6 +1,7 @@
import { Octokit } from "@octokit/core";
import { Api } from "@octokit/plugin-rest-endpoint-methods/dist-types/types";
import { components } from "@octokit/openapi-types/types";
import { context } from "@actions/github";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'context' is defined but never used. @typescript-eslint/no-unused-vars

github-actions bot added a commit that referenced this pull request Aug 28, 2022
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Checks from #640

Check Status URL
Test in Pull Request pass https://github.com/CatChen/accept-to-ship-action/runs/8060185023?check_suite_focus=true

@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2022

This pull request introduces 1 alert when merging 53dc696 into 619d137 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ESLint doesn't pass. Please fix all ESLint issues.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ESLint doesn't pass. Please fix all ESLint issues.

github-actions bot added a commit that referenced this pull request Aug 28, 2022
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Aug 28, 2022
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Checks from #643

Check Status URL
Test in Pull Request pass https://github.com/CatChen/accept-to-ship-action/runs/8060224773?check_suite_focus=true

@github-actions
Copy link
Contributor

Checks from #644

Check Status URL
Test in Pull Request pass https://github.com/CatChen/accept-to-ship-action/runs/8060224774?check_suite_focus=true

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ESLint doesn't pass. Please fix all ESLint issues.

github-actions bot added a commit that referenced this pull request Aug 28, 2022
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Checks from #646

Check Status URL
Test in Pull Request pass https://github.com/CatChen/accept-to-ship-action/runs/8060235390?check_suite_focus=true

@CatChen CatChen merged commit 3d62867 into main Aug 28, 2022
@CatChen CatChen deleted the 08-28-Excluding_self_check_run_and_same_kind_check_runs_when_waiting_for_checks branch August 28, 2022 20:14
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2022

This pull request introduces 1 alert when merging 384311b into 619d137 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

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.

1 participant