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

new github workflow for new label #28279

Merged
merged 49 commits into from
Aug 23, 2023
Merged

new github workflow for new label #28279

merged 49 commits into from
Aug 23, 2023

Conversation

edik24
Copy link
Contributor

@edik24 edik24 commented Jul 19, 2023

Contributing to Cortex XSOAR Content

When a community contribution is done for partner pack - need to get partner to review and approve.
A new label was added to the content repository and a new github workflow that will fail if the "partner-approved" label was not added to the PR.

If the contribution form verified that this is a partner - the label will be added automatically.
If not, reviewer will need to add this label to pass the new workflow

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: link to the issue

Description

A few sentences describing the overall goals of the pull request's commits.

Must have

  • Tests
  • Documentation

Copy link
Contributor

@ShacharKidor ShacharKidor left a comment

Choose a reason for hiding this comment

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

Great work!
Added a few comments.

Copy link
Contributor

@ShacharKidor ShacharKidor left a comment

Choose a reason for hiding this comment

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

Great Work!
I added a comment about the condition in the github action - please have a look.

Last things:

  1. Please add a description to the PR body explaining what we did here.
  2. Please open a new PR from a forked repo to check that handle_new_external_pr work as expected without any errors.

@edik24
Copy link
Contributor Author

edik24 commented Aug 22, 2023

@ShacharKidor
hi, verified the handle_new_external_pr in:
#29131
and updated description

# Conflicts:
#	Utils/github_workflow_scripts/utils.py
edik24 and others added 6 commits August 22, 2023 16:33
# Conflicts:
#	Utils/github_workflow_scripts/utils.py
# Conflicts:
#	Utils/github_workflow_scripts/utils.py
# Conflicts:
#	Utils/github_workflow_scripts/utils.py
# Conflicts:
#	Utils/github_workflow_scripts/utils.py
@edik24 edik24 merged commit 6b3071d into master Aug 23, 2023
@edik24 edik24 deleted the github_workflow_partner branch August 23, 2023 03:55
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Oct 5, 2023
* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Fix all comments after demo

* Fix all comments after demo

* Fix all comments after demo

* fix comments

* Fix all comments after demo

* Fix all comments after demo

* Fix all comments after demo

* Fix all comments after demo

* moving funciton to utils

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py
israelpoli pushed a commit that referenced this pull request May 15, 2024
* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* new github workflow for new label

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Fix all comments after demo

* Fix all comments after demo

* Fix all comments after demo

* fix comments

* Fix all comments after demo

* Fix all comments after demo

* Fix all comments after demo

* Fix all comments after demo

* moving funciton to utils

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py

* Merge branch 'master' into github_workflow_partner

# Conflicts:
#	Utils/github_workflow_scripts/utils.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants