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

Jump to bottom

x/build/cmd/relui: use announcement link from e-mail task in tweet body #47406

Closed
toothrot opened this issue Jul 26, 2021 · 3 comments
Closed

x/build/cmd/relui: use announcement link from e-mail task in tweet body #47406

toothrot opened this issue Jul 26, 2021 · 3 comments
Assignees
@dmitshur
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
Go Release Team
Milestone
Unreleased

Comments

@toothrot
Copy link
Contributor

Instead of requiring user-input, the Tweet Announcement task should fetch the announcement URL from the e-mail task (#47405).
@gopherbot gopherbot added the Builders x/build issues (builders, bots, dashboards) label Jul 26, 2021
@gopherbot gopherbot added this to the Unreleased milestone Jul 26, 2021
@cagedmantis cagedmantis added the NeedsFix The path to resolution is known, but the work has not been done. label Jul 27, 2021
@cagedmantis cagedmantis added this to Planned in Go Release Team Jul 27, 2021
@heschi heschi self-assigned this Aug 3, 2021
@gopherbot
Copy link

Change https://golang.org/cl/339949 mentions this issue: cmd/relui/workflow: basic workflow functionality

gopherbot pushed a commit to golang/build that referenced this issue Aug 16, 2021
@heschi
internal/workflow: add basic workflow functionality
6437f7a 
Creates the workflow package, which runs computation graphs defined by
a declarative API inspired by Beam. See the package documentation for an
overview.

Future work:
- Allow tasks to log, preferably in a streaming way.
- Enable persistence.
- Add whatever additional observability we need to write a pretty UI.

Updates golang/go#47406.

Change-Id: I6f720191a48549cfee3ad6c305399fe9b63edf9d
Reviewed-on: https://go-review.googlesource.com/c/build/+/339949
Trust: Heschi Kreinick <heschi@google.com>
Run-TryBot: Heschi Kreinick <heschi@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
@dmitshur
Copy link
Contributor

@heschi I plan to take this next now that work on #47405 is almost done, which I think is okay with you.

@dmitshur dmitshur assigned dmitshur and unassigned heschi Jun 17, 2022
@dmitshur dmitshur moved this from Planned to In Progress in Go Release Team Jun 21, 2022
@gopherbot
Copy link

Change https://go.dev/cl/413534 mentions this issue: cmd/relui: join announcement mail and tweet tasks in one workflow

Go Release Team automation moved this from In Progress to Done Jun 23, 2022
@dmitshur dmitshur mentioned this issue Jun 24, 2022
Closed
@golang golang locked and limited conversation to collaborators Jun 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@dmitshur dmitshur

Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
Go Release
Archived in project
Go Release Team
Done
Milestone
Unreleased
Development

No branches or pull requests
5 participants
@toothrot @cagedmantis @dmitshur @gopherbot @heschi