question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Target intrinsic order

See original GitHub issue

Some targets should be run before others, to minimize harm in case one of them fails. The order should be determined by ascending impact/cost of reverting ratio. For example, registry should always be run last. We may also want to have an option to disable this order. How to approach this is open for discussion.

For context, see https://github.com/getsentry/sentry-javascript/pull/4171.

Issue Analytics

  • State:open
  • Created 2 years ago
  • Comments:9 (9 by maintainers)

github_iconTop GitHub Comments

1reaction
BYKcommented, Nov 25, 2021

@iker-barriocanal not every target. The current way, the ordered-list definition does what you say and we will keep that way of defining targets for backwards compatibility. What I’m proposing is a new type of target config with a dict where you can optionally set dependencies.

For instance, for the Craft releases themselves, I see on reason why npm, docker, and registry targets to wait for each other. Heck, even the github target can just happen anytime as it has its own artifact, the compiled binary. So, for Craft itself for instance, running all targets in parallel would be the more sensible solution. For any other project that wants to be more conservative, they can keep using the old list-style config or manually depend on things (such as registry depending on others, and github depending on registry etc.)

0reactions
iker-barriocanalcommented, Nov 25, 2021

I understand, but that way will make every target depend on another target, which means executing targets linearly in a specific order and not parallelizing them. Is there something I’m missing?

Read more comments on GitHub >

github_iconTop Results From Across the Web

Intrinsic Wine Co. - Target
Shop Target for a wide assortment of Intrinsic Wine Co.. Choose from Same Day Delivery, Drive Up or Order Pickup. Free standard shipping...
Read more >
Target - Intrinsic Valuation - Apps on Google Play
Intrinsic value is a core concept of value investors seeking to uncover hidden investment opportunities. To calculate intrinsic value, ...
Read more >
Target (TGT) Stock Analysis and Intrinsic Value - YouTube
Target (TGT) Stock Analysis. This video will provide an updated analysis of Target's business, technical analysis and 2 different valuations ...
Read more >
Intrinsic Value Defined and How It's Determined in Investing ...
Intrinsic value is a measure of what an asset is worth. This measure is arrived at by means of an objective calculation or...
Read more >
Target Stock: Undervalued Again (NYSE:TGT) | Seeking Alpha
Target : Same-day services (drive up, Shipt, order pickup) ... Target also seems to be trading below its intrinsic value again.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found