Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

plugins 2: no more plugins #308

Merged
merged 22 commits into from
Dec 2, 2023
Merged

plugins 2: no more plugins #308

merged 22 commits into from
Dec 2, 2023

Conversation

rhelmot
Copy link
Member

@rhelmot rhelmot commented Mar 22, 2023

We begin by discarding all previously held notions of what is good and just. The things that are good and just are the things that work. We must not have the dependency cycle; this is bad. Pyvex cannot be refactored to not use arch, this is bad. Making archinfo not depend on pyvex is a fool's errand, this is bad. Therefore, pyvex must put its finger to its lips and claim, "I do not depend on archinfo".

It's still not good. But it does work.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 23, 2023

Test Results

   103 files  +     93     103 suites  +93   1h 20m 31s ⏱️ + 1h 20m 15s
1 280 tests +1 223  1 242 ✔️ +1 185  38 💤 +38  0 ±0 
1 286 runs  +1 229  1 248 ✔️ +1 191  38 💤 +38  0 ±0 

Results for commit 919a795. ± Comparison against base commit a1fb2a4.

♻️ This comment has been updated with latest results.

@twizmwazin
Copy link
Member

Tests are still using archinfo, fixing those up then plan to merge

@twizmwazin twizmwazin merged commit 0f32e9d into master Dec 2, 2023
17 of 19 checks passed
@twizmwazin twizmwazin deleted the feat/plugins2 branch December 2, 2023 00:16
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.

None yet

2 participants