Skip to content

chore(ci): automatically add area label based on title #1300

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.

Sign up for GitHub

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

Merged
merged 6 commits into from
Jul 15, 2022
Merged

chore(ci): automatically add area label based on title #1300

Changes from 4 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
3105db4
chore(ci): automatically add area label based on title
rubenfonseca Jul 14, 2022
df88965
fix(ci): bad indentation
rubenfonseca Jul 14, 2022
7343c4d
fix(ci): indentation
rubenfonseca Jul 14, 2022
729ae82
fix(ci): indentation
rubenfonseca Jul 14, 2022
4b094c4
fix(ci): typo
rubenfonseca Jul 14, 2022
d5682bf
chore(ci): moved known areas to constants.js
rubenfonseca Jul 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
45 changes: 39 additions & 6 deletions .github/scripts/label_pr_based_on_title.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,24 @@ module.exports = async ({github, context, core}) => {
"deprecated": DEPRECATED_REGEX,
}

const areas = [
"tracer",
"metric",
"utilities",
"logger",
"event_handlers",
"middleware_factory",
"idempotency",
"event_sources",
"feature_flags",
"parameters",
"batch",
"parser",
"validator",
"jmespath_util",
"lambda-layers",
];

// Maintenance: We should keep track of modified PRs in case their titles change
let miss = 0;
try {
Expand All @@ -26,12 +44,27 @@ module.exports = async ({github, context, core}) => {
if (isMatch != null) {
core.info(`Auto-labeling PR ${PR_NUMBER} with ${label}`)

return await github.rest.issues.addLabels({
issue_number: PR_NUMBER,
owner: context.repo.owner,
repo: context.repo.repo,
labels: [label]
await github.rest.issues.addLabels({
issue_number: PR_NUMBER,
owner: context.repo.owner,
repo: context.repo.repo,
labels: [label]
})

const area = matches[2]; // second capture group contains the area
if (areas.indexOf(area) > -1) {
core.info(`Auto-labeling PR ${PR_NUMBER} with area ${area}`);
await github.rest.issues.addLabels({
issue_number: PR_NUMBER,
owner: context.repo.owner,
repo: context.repo.repo,
labels: [`area/${area}`],
});
} else {
core.debug(`'${PR_TITLE}' didn't match any known area.`);
}

return;
} else {
core.debug(`'${PR_TITLE}' didn't match '${label}' semantic.`)
miss += 1
Expand All @@ -42,4 +75,4 @@ module.exports = async ({github, context, core}) => {
return core.notice(`PR ${PR_NUMBER} title '${PR_TITLE}' doesn't follow semantic titles; skipping...`)
}
}
}
}