Skip to content

Conversation

v1v
Copy link
Member

@v1v v1v commented Apr 1, 2025

What does this pull request do?

Use gh cli to add labels
Use our pre-commit composite action: https://github.com/elastic/oblt-actions/tree/main/pre-commit

Related issues

See #2254 (comment)

@v1v v1v added community Issues opened by the community triage Issues awaiting triage labels Apr 1, 2025
@v1v v1v requested a review from kruskall April 1, 2025 10:34
@v1v v1v self-assigned this Apr 1, 2025
@v1v v1v requested review from a team April 1, 2025 10:34
@v1v v1v marked this pull request as ready for review April 1, 2025 10:34
@v1v v1v merged commit ef06a16 into elastic:main Apr 2, 2025
122 checks passed
uses: actions-ecosystem/action-add-labels@v1
with:
labels: agent-python
run: gh issue edit "$NUMBER" --add-label "agent-python"
Copy link
Member

@xrmx xrmx Apr 4, 2025

Choose a reason for hiding this comment

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

@v1v This is not working, NUMBER is empty when I am opening a PR :). See https://github.com/elastic/apm-agent-python/actions/runs/14264122101/job/39982015773?pr=2261

Copy link
Member Author

Choose a reason for hiding this comment

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

Uhm, I'll review it

triage:
runs-on: ubuntu-latest
env:
NUMBER: ${{ github.event.issue.number }}
Copy link
Member Author

Choose a reason for hiding this comment

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

Should be github.event.issue.number || github.event.pull_request.number or something along those lines. I'll review exactly the correct context field

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-python community Issues opened by the community triage Issues awaiting triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants