Skip to content

PXP-595: integrate github pipeline while fetching builds #138

Merged
onimsha merged 25 commits intomainfrom
feat/integrate-github-pipeline
Sep 5, 2023
Merged

PXP-595: integrate github pipeline while fetching builds #138
onimsha merged 25 commits intomainfrom
feat/integrate-github-pipeline

Conversation

@mfauzaan
Copy link
Contributor

@mfauzaan mfauzaan commented Aug 23, 2023

This PR updates the logic of existing command wukong pipeline list and wukong deployment execute.
New logic will fetch from github pipelines and if there is no pipeline found it fallbacks to fetch from jenkins to support the interim migration period

Summary

Ticket: https://mindvalley.atlassian.net/browse/PXP-595

What's Changed

  • Sync graphql schema with new changes
  • Update fetch pipeline to fetch from github pipelines

@mfauzaan mfauzaan changed the title Feat/integrate GitHub pipeline PXP-595: integrate github pipeline to fetch pipelines Aug 24, 2023
@mfauzaan mfauzaan marked this pull request as ready for review August 24, 2023 05:36
@mfauzaan mfauzaan requested review from jk-gan and onimsha August 24, 2023 05:43
@mfauzaan mfauzaan changed the title PXP-595: integrate github pipeline to fetch pipelines PXP-595: integrate github pipeline while fetching builds Aug 28, 2023
@mfauzaan mfauzaan changed the base branch from v1 to main August 30, 2023 03:52
jk-gan
jk-gan previously approved these changes Sep 4, 2023
@sauron-droid
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jk-gan, mfauzaan

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@onimsha onimsha merged commit 08687f0 into main Sep 5, 2023
@sauron-droid sauron-droid deleted the feat/integrate-github-pipeline branch September 5, 2023 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants