Adapt the link mention in the linkedin post content depending on the presence of paper in the PR

Merged
YA
1mo ago
+13-8

When the LinkedIn post workflow is triggered by a merged pull request, the line mentioning the pull request was always
`Added to goldener-research in: `, even when the pull request added no paper (and the post was then about a random
paper of the bibliography).

  • The `Choose the paper` step, which already compares the papers of the base and merge commits, now sets
    `TRIGGER_LINE` through `GITHUB_ENV`:
    • `Added to goldener-research in: ` when the pull request added at least one paper;
    • `Triggered by merging: ` otherwise.
  • The fixed `TRIGGER_LINE` of the job's environment is removed; the prompt and the `Check the post` step are unchanged.
  • Manual runs still get no line.

Tested by running the step's Python locally with and without an added paper, and the YAML parses.

🤖 Generated with Claude Code


Summary by cubic

When a merged pull request triggers the LinkedIn post workflow, the line mentioning the pull request now depends on whether the pull request added papers.

  • "Added to goldener-research in: <url>" when the pull request added at least one paper; "Triggered by merging: <url>" when it added none, since the post is then about a random paper of the bibliography.
  • The Choose the paper step sets TRIGGER_LINE via GITHUB_ENV, replacing the fixed value in the job environment; the prompt and Check the post step are unchanged.
  • Manual runs still get no line.