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

Add python CI #3

Closed
wants to merge 12 commits into from
Closed

Add python CI #3

wants to merge 12 commits into from

Conversation

xiaoyongzhumsft
Copy link
Contributor

No description provided.

@xiaoyongzhumsft xiaoyongzhumsft deleted the snowmanmsft-patch-2 branch February 19, 2022 10:25
blrchen added a commit that referenced this pull request May 5, 2022
* Init react based feathr ui code (#1)

* Enables continues integration and deployment to cloud for ui app (#3)

This PR enables continues integration and deployment to cloud for ui app

- Manually create azure web app under ACE subscription resource group feathr-ui-test
- Create github action workflow to trigger deployment on main branch push or pull request
- Deploy policy
   - Main branch will be deployed to https://thankful-mud-0addb6c00.1.azurestaticapps.net/
   - Each pull request will be deployed to a staging environment, the url will be generated dynamically n each PR's comment.

* Add UI landing page

* Workaround for CI treat warning as error

* Enable client side route in Azure Static Web App

* Code clean up

* Code clean up

* Enable eslint rules

* Update README.md

* Fix CI error

* Add data source mockup page

* Enable Azure AD login

* Complete api integration

* Complete api integration

* Show lineage in feature detail page

* Resolve demo review comments

* Update page title

* Add lineage UI page

* Update Web UI CICD file name and secret name

* Workaround for PR from forked branch can not access main repo's secret

* Workaround for PR from forked branch can not access main repo's secret

* Resolve code review comments

* Test use main repo's secret

* Revert "Test use main repo's secret"

This reverts commit 0ca7f07.

* Remove two more hard codings
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.

2 participants