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

docs: Add Flank output research docs #1644

Merged
merged 2 commits into from
Mar 5, 2021
Merged

Conversation

piotradamczyk5
Copy link
Contributor

@piotradamczyk5 piotradamczyk5 commented Mar 1, 2021

Fixes #1555

Please check flank_output_propositon.md for new output proposition.
flank_current_output.md and gcloud_current_output.md are added for comparsion

Checklist

@github-actions
Copy link
Contributor

github-actions bot commented Mar 1, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@github-actions
Copy link
Contributor

github-actions bot commented Mar 1, 2021

Timestamp: 2021-03-05 13:34:07
Buildscan url for ubuntu-workflow run 624550702
https://gradle.com/s/cez5uir5gggi2

Copy link
Contributor

@pawelpasterz pawelpasterz left a comment

Choose a reason for hiding this comment

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

How about iOS output? It's a bit different from android (in the context of flank).
Also, smart flank output -- are we going to change it somehow?

@jan-goral
Copy link
Contributor

The links in the description doesn't work

@jan-goral
Copy link
Contributor

The links are displaying too often, most of them are redundant and look too verbose. IMO when it comes to the bucket, only the link to the root is important. About firebase, the detailed link could be useful only when tests will finish with some issues, otherwise, we only need one link per matrix.

@piotradamczyk5
Copy link
Contributor Author

The links in the description doesn't work

fixed

@bootstraponline bootstraponline force-pushed the 1555-flank-ouput-research branch 6 times, most recently from acb4d4a to 0069c14 Compare March 3, 2021 15:52
@bootstraponline bootstraponline force-pushed the 1555-flank-ouput-research branch 2 times, most recently from a234a51 to 8bc87bc Compare March 4, 2021 11:17
@piotradamczyk5
Copy link
Contributor Author

How about iOS output? It's a bit different from android (in the context of flank).
Also, smart flank output -- are we going to change it somehow?

For a simple scenario outside output will be similar.
In another not simple cases it should be aligned during implementation

@bootstraponline bootstraponline force-pushed the 1555-flank-ouput-research branch from 8bc87bc to f29e657 Compare March 4, 2021 16:47
@bootstraponline bootstraponline force-pushed the 1555-flank-ouput-research branch from f29e657 to c885319 Compare March 5, 2021 12:17
@bootstraponline bootstraponline force-pushed the 1555-flank-ouput-research branch from c885319 to a75d7ee Compare March 5, 2021 13:29
@mergify mergify bot merged commit f5c8b1c into master Mar 5, 2021
@mergify mergify bot deleted the 1555-flank-ouput-research branch March 5, 2021 13:42
@github-actions github-actions bot locked and limited conversation to collaborators Mar 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve flank output [RESEARCH]
5 participants