-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: Implement SourceHub ACP #2657
base: develop
Are you sure you want to change the base?
Conversation
e957409
to
4005996
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #2657 +/- ##
===========================================
- Coverage 79.18% 79.10% -0.08%
===========================================
Files 319 320 +1
Lines 24256 24468 +212
===========================================
+ Hits 19206 19354 +148
- Misses 3665 3708 +43
- Partials 1385 1406 +21
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 13 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
15f6597
to
d6d6417
Compare
3fe10a0
to
cf8ac0d
Compare
0414ffe
to
fd25896
Compare
4d7d44c
to
b98d1b8
Compare
e831117
to
c6955cf
Compare
This is needed by a couple of acp packages when running against source-hub
b5493d9
to
88a072d
Compare
88a072d
to
74b4383
Compare
Relevant issue(s)
Resolves #2366
Description
Implements SourceHub ACP.
Whilst the time out for a single package has been increased, the CI cost of the new jobs remains about the same as when running with
source-hub
acp, the integration test harness will skip all non-acp tests.Warnings
This PR introduces a new dependency to be manually installed by anyone running the tests locally. It is noted in the contributors readme. It can be removed once a SourceHub test harness thing has been built allowing us to stop using the SourceHub CLI in our tests.