Skip to content

Conversation

@twsouthwick
Copy link
Member

@twsouthwick twsouthwick commented Nov 19, 2024

With the pipelining added in 3.0.1 used internally for creating packages, it caused any initializing of the package (i.e. from FlatOpc) to be repeated on clones which we don't want. This fixes the templates (both FlatOpc and document templates) from not being captures in the package creation but kept separate for an initialization step.

Fixes #1814

@github-actions
Copy link

github-actions bot commented Nov 19, 2024

Test Results

    70 files  ± 0      70 suites  ±0   1h 7m 32s ⏱️ -1s
 2 036 tests + 3   2 033 ✅ + 3   3 💤 ±0  0 ❌ ±0 
32 358 runs  +36  32 322 ✅ +36  36 💤 ±0  0 ❌ ±0 

Results for commit 2984934. ± Comparison against base commit 4fae05d.

♻️ This comment has been updated with latest results.

mikeebowen
mikeebowen previously approved these changes Nov 19, 2024
With the pipelining added in 3.0.1 used internally for creating packages, it caused any initializing of the package (i.e. from FlatOpc) to be repeated on clones which we don't want. This fixes it for the near term to unblock users, but will be followed with a consolidation of the uses of IPackageInitialzier.
@twsouthwick twsouthwick force-pushed the fix-flatopc-and-cloning branch from 0f4cf0c to 2984934 Compare November 19, 2024 19:28
@twsouthwick twsouthwick marked this pull request as ready for review November 19, 2024 19:37
@twsouthwick twsouthwick enabled auto-merge (squash) November 19, 2024 19:39
@twsouthwick
Copy link
Member Author

@mikeebowen looks like it rejected your approval after I pushed some updates

@twsouthwick twsouthwick merged commit 585bd6a into main Nov 19, 2024
22 checks passed
@twsouthwick twsouthwick deleted the fix-flatopc-and-cloning branch November 19, 2024 19:55
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.

Flat OPC and cloning features are broken

4 participants