Skip to content

Commit

Permalink
Refactor tasks to not fork, and always run in-process of the main exe…
Browse files Browse the repository at this point in the history
…cution pipeline (#1613)

* Check that Deno v1.23.x is used when building Taqueria

* Temporary commit

* Added means of registering internal tasks, and to process those tasks inside the same pipeline as tasks provided by plugins.

* Segmented global and internal tasks

* Ensure that internal tasks are invoked

* Fixed memory leak in test/e2e/utils

* Fix broken test due to help output changing

* Fixed failing test spec due to help output changes

* Re-add afterAll hook to cleanup

* Fix broken tests due to help output

* Fixed help output for contract types test spec

* Committing work thus far

* Accomodate aliases when determining whether a task is runninng

* Fixed problem with taq not outputting in non-taquified directory

* Skip pinata help output tests

* Skipping some tests due to help output changing

* Assure that we can identify whether tasks are being run which have spaces in their task name

* Fixed issue with parsedArgs._ being modified after parseArgs is run

* Ensure that detection for whether a given task is running is working

* Skip some tests that fail due to change in help output

* Add missing aliases to some registered tasks

* Skipped more tests due to changes in help text output

* Fixed issues with failing tests in metadata test suite due to legacy code

* Fixed broken ligo test due to skipping a previous test

* Removed debugger statements

* Fixed typo

* Added link to issue for todo item

* Addressed PR comments

Co-authored-by: Houston <[email protected]>
  • Loading branch information
Michael Weichert and hu3man authored Dec 13, 2022
1 parent 2a345ed commit 61d1ca0
Show file tree
Hide file tree
Showing 30 changed files with 1,011 additions and 787 deletions.
13 changes: 9 additions & 4 deletions analytics.ts
Original file line number Diff line number Diff line change
Expand Up @@ -68,10 +68,15 @@ export const inject = (deps: UsageAnalyticsDeps) => {
return taqResolve('');
}
}),
mapRej(() =>
option === OPT_IN
? 'The command "taq opt-in" is ignored as this might be the first time running Taqueria...'
: 'The command "taq opt-out" is ignored as this might be the first time running Taqueria...'
mapRej(previous =>
TaqError.create({
kind: 'E_OPT_IN_WARNING',
msg: option === OPT_IN
? 'The command "taq opt-in" is ignored as this might be the first time running Taqueria...'
: 'The command "taq opt-out" is ignored as this might be the first time running Taqueria...',
previous,
context: option,
})
),
);

Expand Down
Loading

0 comments on commit 61d1ca0

Please sign in to comment.