Skip to content

Minor tweaks to issue-opening bot - #178

Open
larsoner wants to merge 4 commits into
scientific-python:mainfrom
larsoner:stale-wheel-followup
Open

Minor tweaks to issue-opening bot#178
larsoner wants to merge 4 commits into
scientific-python:mainfrom
larsoner:stale-wheel-followup

Conversation

@larsoner

@larsoner larsoner commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Leaving as draft for a few days in case there are other tweaks now that we have issues in the open. But from looking and using it:

  1. Add job-level annotations (I was on a job page and saw nothing -- redundant with the action summary but puts in two places which helps discoverability)
  2. Add 🤖 to last line of opened issue to make it visually clearer it's a bot-opened issue
  3. Tweak dry-run wording (it says it opened the issue... but it would open the issue)
  4. Adds note that the "last updated" on the Anaconda site is not the last uploaded wheel -- deleting an old wheel counts as an update (and so the "last updated" date reflects the deletion date, not the most-recent-upload date)

@larsoner

larsoner commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

I'll take care of #181 here once we converge on how to publish the dates!

@larsoner
larsoner force-pushed the stale-wheel-followup branch from ff4323e to 5c15118 Compare September 3, 2026 18:06
@larsoner
larsoner marked this pull request as ready for review September 3, 2026 18:06
@larsoner

larsoner commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

@bsipocz locally doing:

$ export ISSUE_OPENER_TOKEN=$(gh auth token) GITHUB_TOKEN=$(gh auth token)
$ uv run --with-requirements tools/requirements.txt tools/check_stale_wheels.py --dry-run --html site/index.html
Acting as larsoner
$ open site/index.html

I see:

Screenshot 2026-09-03 at 14 13 30

Ready for review/merge from my end!

Comment thread .github/workflows/stale-wheels.yml Outdated
run: |
uv run --with-requirements tools/requirements.txt tools/check_stale_wheels.py ${{ inputs.dry_run && '--dry-run' || '' }}
uv run --with-requirements tools/requirements.txt tools/check_stale_wheels.py \
--html site/index.html ${{ inputs.dry_run && '--dry-run' || '' }}

@matthewfeickert matthewfeickert Sep 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

At the moment the deployment is

$ tree ./site/
./site/
├── index.html
└── status.css

1 directory, 2 files

To avoid moving the page later if we want to use the top level for different information, can we have this either deploy to site/status.html with a landing page that routes there? If that's too much work to get something up today can we just copy index.html to status.html until we do something else so we know that status.html will work in the future?

@larsoner

larsoner commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

I'll see if I can get the README as the landing page with a pointer to the status page

@larsoner

larsoner commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

Okay now index.html:

Screenshot 2026-09-03 at 15 57 17

With a pointer at the top to status.html:

Screenshot 2026-09-03 at 15 57 20

@matthewfeickert matthewfeickert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @larsoner

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.

Publish stale wheels report as a status page website

2 participants