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

Added support of aws codepipelines #1192

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

subhajit20
Copy link

Problem

[Provide a brief description of the problem that you are trying to solve with this pull request.]
Added support of aws codepipelines

Issue - #1176

Solution

[Provide a detailed explanation of the solution you are proposing, including any relevant code snippets or screenshots.]

Changes Made

  • [List the changes you made in this pull request, including any new features or bug fixes.]

How to Test

[Provide instructions on how to test the changes you made, including any relevant details like configuration steps or data to be used for testing.]

Screenshots

[Include screenshots, if relevant, to help reviewers understand the changes you made.]

Notes

[Any additional notes or information that you would like to share with the reviewers.]

Checklist

  • Code follows the contributing guidelines
  • Changes have been thoroughly tested
  • Documentation has been updated, if necessary
  • Any dependencies have been added to the project, if necessary

Reviewers

@[username of the reviewer]

Copy link
Collaborator

@Azanul Azanul left a comment

Choose a reason for hiding this comment

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

Also, log non fatal errors as warnings

providers/aws/CodePipelines/GetPipelines.go Outdated Show resolved Hide resolved
providers/aws/CodePipelines/GetPipelines.go Outdated Show resolved Hide resolved
@Azanul
Copy link
Collaborator

Azanul commented Nov 13, 2023

@subhajit20 CI is failing

@subhajit20
Copy link
Author

@subhajit20 CI is failing

hey can you help out? I am bit confused

@Azanul
Copy link
Collaborator

Azanul commented Nov 13, 2023

@subhajit20 CI is failing

hey can you help out? I am bit confused

Try to run in your local, it'll throw the errors you need to fix

@AvineshTripathi
Copy link
Collaborator

Hey @subhajit20 were you able to resolve the issue?

@subhajit20
Copy link
Author

Hey @subhajit20 were you able to resolve the issue?

No, I am not... I have not been able to generate credentials also.

@AvineshTripathi
Copy link
Collaborator

Acc to the error you can just do go get github.com/aws/aws-sdk-go-v2/service/pipeline and error should be resolved

@Azanul
Copy link
Collaborator

Azanul commented Dec 13, 2023

@subhajit20 were you able to resolve the error?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants