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.

Sign up for GitHub

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

Jump to bottom

Update CONTRIBUTING.md #180

Merged
merged 14 commits into from
Jun 28, 2024
Merged

Update CONTRIBUTING.md #180

merged 14 commits into from
Jun 28, 2024

Conversation

tylertitsworth
Copy link
Contributor

@tylertitsworth tylertitsworth commented Jun 26, 2024 •
edited
Loading

Description

Accounts for the new CI/CD requirements.

Related Issue

MLOPS-2015

Changes Made

  • The code follows the project's coding standards.
  • No Intel Internal IP is present within the changes.
  • The documentation has been updated to reflect any changes in functionality.

Validation

  • I have tested any changes in container groups locally with test_runner.py with all existing tests passing, and I have added new tests where applicable.

@tylertitsworth tylertitsworth added the WIP Work in Progress label Jun 26, 2024
@tylertitsworth tylertitsworth self-assigned this Jun 26, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 26, 2024 •
edited
Loading

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

tylertitsworth added 10 commits June 27, 2024 13:56
update contributing docs
a82197f 
Signed-off-by: tylertitsworth <[email protected]>
Add Coveralls (#166)
14b3854 
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: tylertitsworth <[email protected]>
Update README.md
a1f385c 
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: tylertitsworth <[email protected]>
add intel dco text
aaa9405 
Signed-off-by: tylertitsworth <[email protected]>
fix license link
97644d6 
Signed-off-by: tylertitsworth <[email protected]>
remove redundant line
017689f 
Signed-off-by: tylertitsworth <[email protected]>
Add push event to test runner CI for unit tests base build (#184)
14a3b79 
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: tylertitsworth <[email protected]>
Update CODEOWNERS priority (#182)
6027a2b 
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: tylertitsworth <[email protected]>
Remove perf test text (#181)
598e6b7 
Signed-off-by: tylertitsworth <[email protected]>
Update test-runner-ci.yaml
5d39dd0 
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: tylertitsworth <[email protected]>
@tylertitsworth tylertitsworth force-pushed the tylertitsworth/infra-contrib-updates branch from 993593f to 5d39dd0 Compare June 27, 2024 20:57
@coveralls-official Coveralls Official
Copy link

coveralls-official bot commented Jun 27, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9703484349

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 88.485%
Totals Coverage Status
Change from base Build 9703399524: 0.0%
Covered Lines: 146
Relevant Lines: 165
💛 - Coveralls

Update CONTRIBUTING.md
4e54460 
Signed-off-by: Tyler Titsworth <[email protected]>
@tylertitsworth tylertitsworth added Review and removed WIP Work in Progress labels Jun 27, 2024
@tylertitsworth tylertitsworth added this pull request to the merge queue Jun 28, 2024
sharvil10
sharvil10 approved these changes Jun 28, 2024
View reviewed changes
Copy link
Contributor

@sharvil10 sharvil10 left a comment

Choose a reason for hiding this comment

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

LGTM

@tylertitsworth tylertitsworth merged commit f707fdc into main Jun 28, 2024
2 of 3 checks passed
@tylertitsworth tylertitsworth deleted the tylertitsworth/infra-contrib-updates branch June 28, 2024 16:42
ma-pineda pushed a commit that referenced this pull request Jun 28, 2024
@ma-pineda
Update CONTRIBUTING.md (#180)
e386555 
Signed-off-by: tylertitsworth <[email protected]>
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: ma-pineda <[email protected]>
ma-pineda pushed a commit that referenced this pull request Jun 28, 2024
@ma-pineda
Update CONTRIBUTING.md (#180)
37eaddd 
Signed-off-by: tylertitsworth <[email protected]>
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: ma-pineda <[email protected]>
ma-pineda pushed a commit that referenced this pull request Jun 28, 2024
@ma-pineda
Update CONTRIBUTING.md (#180)
d27f2e9 
Signed-off-by: tylertitsworth <[email protected]>
Signed-off-by: Tyler Titsworth <[email protected]>
Signed-off-by: ma-pineda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jitendra42 jitendra42 jitendra42 approved these changes

@sharvil10 sharvil10 sharvil10 approved these changes

Assignees

@tylertitsworth tylertitsworth

Labels
Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tylertitsworth @sharvil10 @jitendra42