Skip to content

Navigation Menu

Sign in
Sign up

Test source distributions outside the checkout - #1770

Open
ryux1 wants to merge 1 commit into
collective:main from
ryux1:fix/1708-test-built-packages
Open

Test source distributions outside the checkout #1770
ryux1 wants to merge 1 commit into
collective:main from
ryux1:fix/1708-test-built-packages

Conversation

@ryux1

@ryux1 ryux1 commented Sep 6, 2026
edited
Loading

Copy link
Copy Markdown

Linked issue

Description

Source distributions omitted generate_windows_to_olson_mapping.py, so two tests shipped in the archive failed outside a repository checkout. This includes the required file and adds an isolated CI job that downloads, unpacks, and tests the built source archive. Release jobs now depend on that test. The isolated result changed from 2 failures and 18,364 passes to 18,366 passes.

Checklist

  • I added a change log entry, following the instructions in all subsections under Change log requirements.
  • I followed icalendar's Artificial intelligence policy and disclosed my Responsible AI use in my commit messages, if applicable.
  • I added or updated tests, if applicable.
  • I ran and ensured all tests pass locally by following Run tests.
  • I added or edited documentation as necessary, both as docstrings to be rendered in the API documentation and narrative documentation, following the Style guide.

AI use: OpenAI Codex based on GPT-5 was used to investigate the artifact layout, draft the workflow and packaging changes, and plan validation. I reviewed the changes and ran the full tests locally.
@github-actions github-actions Bot added the ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding. label Sep 6, 2026

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

This pull request did not pass quality checks and AI use is suspected. Please review Contribute and make any necessary amendments.

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Profile summary:

GitHub user: ryux1
🟢 No concerns found with user's profile.
🟢 No concerns found with recent PR activity.
🟢 No concerns found with recent issue activity.
For a more detailed report, run `gh-profiler ryux1`.
Full profile
GitHub user: ryux1
🟢 No concerns found with user's profile.
 🟢 Account age: 2 years
 🟢 Profile information:
 name: Ryu
 company: Earth
 location: Solar System,
 bio:
 const ryu={role:"Systems architect",focus:"Networks+Scale",guard:"Integrity+Stability",exec:()=> "precision"}
 Empty fields: blog, email
🟢 No concerns found with recent PR activity.
 51 PRs opened in the last 21 days.
 9 opened against repos the user owns.
 0 opened against repos in publicly associated orgs.
 42 opened against external repos.
 🟢 4 of 42 external PRs closed without merging in the last 21 days.
🟢 No concerns found with recent issue activity.
 4 new issues opened in the last 21 days.
 3 opened in repos the user owns.
 0 opened in repos in publicly associated orgs.
 1 opened in external repos.
 🟢 0 external issues closed as NOT_PLANNED.
 🟢 0 external issues opened with the same title.

Copy link
Copy Markdown

Coverage Status

coverage: 97.562%. remained the same — ryux1:fix/1708-test-built-packages into collective:main

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

Reviewers

@niccokunzmann niccokunzmann Awaiting requested review from niccokunzmann niccokunzmann is a code owner
@stevepiercy stevepiercy Awaiting requested review from stevepiercy stevepiercy is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] missing files for tests

1 participant

AltStyle によって変換されたページ (->オリジナル) /