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

Skip PRs from the triage process #12470

Merged
merged 3 commits into from Dec 18, 2017

Conversation

adelinamart
Copy link
Contributor

Fixes #12434
Added more batches also since the number of issues grew.

Copy link
Contributor

@rsimha rsimha left a comment

Choose a reason for hiding this comment

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

A few minor comments below.

@@ -67,7 +67,7 @@ const BIWEEKLY_DAYS = 14;
// days for quarterly updates
const QUARTERLY_DAYS = 89;
// we need around 10 batches to get 1k issues
Copy link
Contributor

Choose a reason for hiding this comment

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

Update this comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

// if an issue is an pull request will skip it
if (pullRequest) {
if (isDryrun) {
util.log(util.colors.red(issue.number + ' is pull request'));
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: "is a pull request"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done. Thank you!

@@ -136,6 +136,13 @@ function updateGitHubIssues() {
const issueLastUpdate = issue.updated_at;
let biweeklyUpdate = true;
let quartelyUpdate = true;
// if an issue is an pull request will skip it
Copy link
Contributor

Choose a reason for hiding this comment

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

if an issue is a pull request, we'll skip it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@dreamofabear
Copy link

Thanks for fixing this!

@ampprojectbot ampprojectbot added this to the Pending Triage milestone Dec 18, 2017
@dreamofabear dreamofabear merged commit 7ac5140 into ampproject:master Dec 18, 2017
gzgogo pushed a commit to gzgogo/amphtml that referenced this pull request Jan 26, 2018
* fixes for automation

* fixes for automation

* fixes for automation with review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants