Skip to content

Test source distributions outside the checkout - #1770

Draft
ryux1 wants to merge 1 commit into
collective:mainfrom
ryux1:fix/1708-test-built-packages
Draft

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

Conversation

@ryux1

@ryux1 ryux1 commented Sep 6, 2026

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 source-distribution result changes from two failures to a passing suite. After rebasing onto the current main, the full checkout suite passes with 18,382 tests, 24 skips, and 526 expected failures.

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.

Prepared with OpenAI Codex assistance. The contributor's personal review, responsibility, and understanding attestation remains pending; the PR will stay in draft until that is complete.

@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

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

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.

@coveralls-official

coveralls-official Bot commented Sep 6, 2026

Copy link
Copy Markdown

Coverage Status

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

@ryux1
ryux1 force-pushed the fix/1708-test-built-packages branch from 27b3a89 to 5467a65 Compare September 7, 2026 23:31
@ryux1
ryux1 marked this pull request as draft September 7, 2026 23:31
AI use: OpenAI Codex based on GPT-5 was used to investigate the artifact layout, draft the workflow and packaging changes, and plan validation. Automated validation included the full test suite. Personal human review and responsibility attestation are pending.
@ryux1
ryux1 force-pushed the fix/1708-test-built-packages branch from 5467a65 to bb465df Compare September 8, 2026 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] missing files for tests

1 participant