Skip to content

Add dcp_trigger_build with GitHub links and log tailing#2265

Merged
alexrichey merged 2 commits intomainfrom
ar-trigger-gha-build
Mar 4, 2026
Merged

Add dcp_trigger_build with GitHub links and log tailing#2265
alexrichey merged 2 commits intomainfrom
ar-trigger-gha-build

Conversation

@alexrichey
Copy link
Contributor

@alexrichey alexrichey commented Mar 4, 2026

Never Futz with the GHA build dataset dialogue ever again!

  • Add direct GitHub Actions run URL after dispatching workflow
  • Automatically tail logs by default for immediate feedback
  • Add --no-tail flag to skip log tailing when desired

Usage examples:

# From product directory (uses DATASET from direnv) 
cd products/pluto && dcp_trigger_build

# Explicit dataset
dcp_trigger_build pluto

# With custom recipe and build note
dcp_trigger_build pluto my_recipe "test build"

# Skip log tailing 
dcp_trigger_build pluto --no-tail

Note: When in a products/* directory with direnv enabled, the DATASET environment variable is automatically set, so you can run the command without any arguments.

- Add direct GitHub Actions run URL after dispatching workflow
- Automatically tail logs by default for immediate feedback
- Add --no-tail flag to skip log tailing when desired

Usage examples:
  # From product directory (uses DATASET from direnv)
  cd products/pluto && dcp_trigger_build

  # Explicit dataset
  dcp_trigger_build pluto

  # With custom recipe and build note
  dcp_trigger_build pluto my_recipe "test build"

  # Skip log tailing
  dcp_trigger_build pluto --no-tail

Note: When in a products/* directory with direnv enabled, the DATASET
environment variable is automatically set, so you can run the command
without any arguments.
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.69%. Comparing base (19de2d5) to head (15de73f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@damonmcc damonmcc left a comment

Choose a reason for hiding this comment

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

nice!

the github CLI seems simple enough to do this via bash. in general though, fair to say we still prefer python over bash for custom scripts? we don't get the bonus of adding those to our path though like we do with bash scripts in bash/bin/

@alexrichey
Copy link
Contributor Author

nice!

the github CLI seems simple enough to do this via bash. in general though, fair to say we still prefer python over bash for custom scripts? we don't get the bonus of adding those to our path though like we do with bash scripts in bash/bin/

@damonmcc It's mostly about auth, and that the CLI enables you to do the auth flow and store the results. Whereas with Python I suppose we'd have to store some creds in the .env or something.

@alexrichey alexrichey merged commit fef4e7f into main Mar 4, 2026
26 checks passed
@alexrichey alexrichey deleted the ar-trigger-gha-build branch March 4, 2026 20:52
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.

2 participants