-
Notifications
You must be signed in to change notification settings - Fork 5
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
chore: run appinspect api on release branches #376
base: develop
Are you sure you want to change the base?
chore: run appinspect api on release branches #376
Conversation
@@ -873,7 +873,7 @@ jobs: | |||
if: | | |||
!cancelled() && | |||
needs.build.result == 'success' && | |||
( github.base_ref == 'main' || github.ref_name == 'main' ) | |||
( github.base_ref == 'main' || github.ref_name == 'main' || startsWith(github.ref_name, 'release/') ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to check for base_ref as well when it is a PR to release branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dkvashninsplunk Added
Hi @mgrandys-splunk, could you add some link with test to the description? |
10da034
to
af6ad37
Compare
af6ad37
to
bab513b
Compare
@kdoroszko-splunk I've updated the description with 3 test cases for MSCS TA pipeline |
Description
Run
appinspect api
job forrelease/
branches inbuild-test-release
pipelineTicket: https://splunk.atlassian.net/browse/ADDON-77779
Tested in the following cases where appinspect api job is:
Checklist
README.md
has been updated or is not requiredTesting done
(for each selected checkbox, the corresponding test results link should be listed here)