Skip to content

Conversation

@ksaric
Copy link
Contributor

@ksaric ksaric commented Aug 2, 2018

https://iohk.myjetbrains.com/youtrack/issue/TSD-97

Removing the tag after the debugger run.
Required for the next debugger run.

in all (`elem` responseTags) mergedTags
-- in summary, the response tags have the debuggers `analyzed-by-script-version` tag AND
-- they remove the `to_be_analysed` tag AND they are unique.
in (filter (/= renderTicketStatus ToBeAnalyzed) . nub $ renderTicketStatus AnalyzedByScriptV1_3 : mergedTags) === responseTags
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The good thing about this is this will fail once we increase the tag version.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, thank you for letting me know! I'm be careful about it

Copy link
Contributor

@HirotoShioi HirotoShioi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed it. LGTM!

in all (`elem` responseTags) mergedTags
-- in summary, the response tags have the debuggers `analyzed-by-script-version` tag AND
-- they remove the `to_be_analysed` tag AND they are unique.
in (filter (/= renderTicketStatus ToBeAnalyzed) . nub $ renderTicketStatus AnalyzedByScriptV1_3 : mergedTags) === responseTags
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, thank you for letting me know! I'm be careful about it

@HirotoShioi HirotoShioi merged commit 3c76070 into develop Aug 3, 2018
@HirotoShioi HirotoShioi deleted the ksaric/TSD-97 branch October 2, 2018 03:36
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.

3 participants