Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Ignore 'registry_utils' in test coverage#907

Merged
xiaoyongzhu merged 1 commit into
feathr-ai:mainfeathr-ai/feathr:mainfrom
enya-yx:enya/cov-ignoreenya-yx/feathr:enya/cov-ignoreCopy head branch name to clipboard
Dec 9, 2022
Merged

Ignore 'registry_utils' in test coverage#907
xiaoyongzhu merged 1 commit into
feathr-ai:mainfeathr-ai/feathr:mainfrom
enya-yx:enya/cov-ignoreenya-yx/feathr:enya/cov-ignoreCopy head branch name to clipboard

Conversation

@enya-yx
Copy link
Copy Markdown
Collaborator

@enya-yx enya-yx commented Dec 8, 2022

Description

  • Add 'registry_utils' as one of files to be ignored in test coverage report since it's only be applied by '_feature_registry_purview' which is deparacated.

How was this PR tested?

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

@enya-yx enya-yx added the safe to test Tag to execute build pipeline for a PR from forked repo label Dec 8, 2022
@xiaoyongzhu xiaoyongzhu merged commit bafff04 into feathr-ai:main Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xiaoyongzhu xiaoyongzhu xiaoyongzhu approved these changes

@blrchen blrchen blrchen approved these changes

Assignees

No one assigned

Labels

safe to test Tag to execute build pipeline for a PR from forked repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@enya-yx @xiaoyongzhu @blrchen
Morty Proxy This is a proxified and sanitized view of the page, visit original site.