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 license, release, docs, python api ref badges with shields img #357

Merged
merged 1 commit into from
Jun 14, 2022

Conversation

Yuqing-cat
Copy link
Collaborator

This PR includes 4 essential badges:
image

Other information like unit test status or Issue/PR status can also be summarized in this format:
image
To keep content clean, I prefer not to put them in the main page. However, it could be useful for contributors and approvers. So we may consider to put them in dev doc page or some where else.
If needed, I can add them in a separate PR.

@Yuqing-cat Yuqing-cat added the feature New feature or request label Jun 14, 2022
@Yuqing-cat Yuqing-cat self-assigned this Jun 14, 2022
@Yuqing-cat Yuqing-cat linked an issue Jun 14, 2022 that may be closed by this pull request
@Yuqing-cat Yuqing-cat added documentation Improvements or additions to documentation and removed feature New feature or request labels Jun 14, 2022
Copy link
Collaborator

@jainr jainr left a comment

Choose a reason for hiding this comment

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

This is useful, I suggest putting it in the bottom of the landing page that is where usually badges live but don't have a strong preference one way or other.

@xiaoyongzhu xiaoyongzhu merged commit d2bb61a into feathr-ai:main Jun 14, 2022
bozhonghu pushed a commit that referenced this pull request Jun 15, 2022
* main:
  Fixing purview test issues and improve performance (#350)
  [feathr] Add product_recommendation advanced sample (#348)
  obejectId query cmd update (#360)
  add license, release, docs, python api ref badges with shields img (#357)
  quick fix the 404 not found in read me link (#355)
  Python SQL Registry (#311)
  enable JWT token param in frontend API calls (#337)
  Optimize environment variable behavior (#333)
  Adding better warning message to let user know that config file is missing and they need to set env parameters. (#347)
  Feature Monitoring (#330)
  Windoze/211 maven submission (#334)
  Windoze/211 maven submission (#334)
  Windoze/211 maven submission (#334)
  Fix Synapse quickstart link (#346)
  Show feature details when click feature in lineage graph (#339)
  Update pull_request_push_test.yml
  Update UI README for how to create overrides for local development (#335)
  Update databricks quick start experience (#217)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add badges on top of the main page for better experience
3 participants