Skip to content

Conversation

@qwordy
Copy link
Member

@qwordy qwordy commented Sep 25, 2020

Description

  • Update pipeline run title.
  • Generate index.html of testing results.
  • Design a unique representation of a pipeline run.
  • Make storage account container public.
  • Fix a tiny DB bug.
  • Fix pipe not close problem.

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@qwordy qwordy self-assigned this Sep 25, 2020


def render(data, container_url, testdata, USER_REPO, USER_BRANCH, COMMIT_ID, USER_LIVE):
content = """
Copy link
Member

Choose a reason for hiding this comment

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

how about extract it to a seperate file?

Copy link
Member Author

Choose a reason for hiding this comment

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

144 lines. Short file. Don't worry.

Copy link
Member Author

Choose a reason for hiding this comment

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

https://jinja.palletsprojects.com/en/2.11.x/ is a good choice. But it costs more time to write code. The current content is simple. If it gets complex enough, we can use template engine.

@qwordy qwordy merged commit 2067936 into Azure:dev Sep 27, 2020
Juliehzl pushed a commit to Juliehzl/azure-cli that referenced this pull request Sep 30, 2020
* fix a pipe bug

* fix a db bug

* history

* log

* history

* Update pipeline run title

* Generate index.html of HTML files of testing results. Design a unique representation of a pipeline run.

* Make storage account container public

* style

* fix a minor bug

* upload index.html

* pip install requests

* index.html

* index.html

* index.html

* resolve comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants