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

Add GitHub action for issue triaging script #14691

Merged
merged 1 commit into from
May 25, 2021
Merged

Conversation

adbridge
Copy link
Contributor

Summary of changes

This PR adds a GitHub action to perform the issue triaging feature each time a new issue is created or an existing issue
reopened.

Impact of changes

Migration actions required

Documentation


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[x] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers


@adbridge adbridge requested review from 0xc0170 and a team May 21, 2021 14:52
@ciarmcom ciarmcom added the release-type: patch Indentifies a PR as containing just a patch label May 21, 2021
@ciarmcom ciarmcom requested a review from a team May 21, 2021 15:00
@ciarmcom
Copy link
Member

@adbridge, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

- name: Checkout repo
uses: actions/checkout@v2

- name: Python stuff
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- name: Python stuff
- name: Install python env

with:
python-version: 3.8

- name: Checkout mbed-os-scripts repo
Copy link
Contributor

Choose a reason for hiding this comment

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

as t his is yaml, we can add a comment the repository is private.

A question will be if we can move the script to a public repo. I would say this one should be part of this repository, we can take that separately.

@mergify mergify bot dismissed 0xc0170’s stale review May 21, 2021 15:35

Pull request has been modified.

@adbridge
Copy link
Contributor Author

@0xc0170 comments addressed

@ciarmcom ciarmcom requested a review from a team May 21, 2021 16:00
@ciarmcom
Copy link
Member

@adbridge, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

@ciarmcom
Copy link
Member

@adbridge, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

@adbridge
Copy link
Contributor Author

CI started

- name: Checkout mbed-os-scripts repo (This repo is currently private)
uses: actions/checkout@v2
with:
repository: ARMmbed/mbed-os-ci-scripts
Copy link
Contributor

Choose a reason for hiding this comment

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

This works fine even with this repo being private?

Copy link
Contributor

Choose a reason for hiding this comment

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

Seems so

- name: Checkout mbed-os-scripts repo (This repo is currently private)
uses: actions/checkout@v2
with:
repository: ARMmbed/mbed-os-ci-scripts
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems so

@adbridge adbridge merged commit 2ea0386 into ARMmbed:master May 25, 2021
jobs:
build:
runs-on: ubuntu-latest

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

found a few spaces here while checking git diff for other stuff

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.

6 participants